bullet_train-outgoing_webhooks 1.0.0 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '07187a561c9a79ccbf549abeebb9698b0153fe826cc83a508250ae80b9cf9410'
-  data.tar.gz: 50778e5e8d25388c414bae632db09c364a2612f8f089a54eddbf13c0bd570fdd
+  metadata.gz: e4186cf970999d91997f2410efeba593f422788cec8452978544796176d7ea47
+  data.tar.gz: 9700fff07690d45d249e966e29cd233f894b0f77fbffbc4cacf47a637ebb7be6
 SHA512:
-  metadata.gz: 2ec9dab63edb091f2776fef944ade249d31e85bc79106ba5bbbc0fdcbffdcd8db8acb596815aa4b393a8cd84c9029d08f86b9ca5e62d836ba817877671a21aac
-  data.tar.gz: e2a23369f030b68a7aa23411ed4c9281b6b4afd4ddf86cc33003bd3168dab82441b7017044fbbfa757f93a164d50c77ef22f8a5fda8829f15888ad5b51f413a7
+  metadata.gz: 7b971859d534310367961541028daef0b95286d02d1bbe202ac954228ed092dacf6180277193d8b4822547838c20d3d60a7efb0eab1e56160eecf48fe7ac6606
+  data.tar.gz: 268648e2e285ca1398f4b526529c6a7de6a8e8efd461e73fc68003dd579fb75a335a77d3b461efd9d46b906aebc93bdd0f3fdbd6f8191dd60774deada233c491

data/MIT-LICENSE

@@ -0,0 +1,20 @@
+Copyright 2022 Andrew Culver
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -1,43 +1,28 @@
 # BulletTrain::OutgoingWebhooks
+Short description and motivation.
 
-Welcome to your new gem! In this directory, you'll find the files you need to be able to package up your Ruby library into a gem. Put your Ruby code in the file `lib/bullet_train/outgoing_webhooks`. To experiment with that code, run `bin/console` for an interactive prompt.
-
-TODO: Delete this and the text above, and describe your gem
+## Usage
+How to use my plugin.
 
 ## Installation
-
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'bullet_train-outgoing_webhooks'
+gem "bullet_train-outgoing_webhooks"
 ```
 
 And then execute:
-
-    $ bundle install
+```bash
+$ bundle
+```
 
 Or install it yourself as:
-
-    $ gem install bullet_train-outgoing_webhooks
-
-## Usage
-
-TODO: Write usage instructions here
-
-## Development
-
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake test` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
-
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+```bash
+$ gem install bullet_train-outgoing_webhooks
+```
 
 ## Contributing
-
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/bullet_train-outgoing_webhooks. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/[USERNAME]/bullet_train-outgoing_webhooks/blob/main/CODE_OF_CONDUCT.md).
+Contribution directions go here.
 
 ## License
-
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
-
-## Code of Conduct
-
-Everyone interacting in the BulletTrain::OutgoingWebhooks project's codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/bullet_train-outgoing_webhooks/blob/main/CODE_OF_CONDUCT.md).

data/Rakefile

@@ -1,14 +1,8 @@
-# frozen_string_literal: true
+require "bundler/setup"
 
-require "bundler/gem_tasks"
-require "rake/testtask"
-
-Rake::TestTask.new(:test) do |t|
-  t.libs << "test"
-  t.libs << "lib"
-  t.test_files = FileList["test/**/test_*.rb"]
-end
+APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
+load "rails/tasks/engine.rake"
 
-require "standard/rake"
+load "rails/tasks/statistics.rake"
 
-task default: %i[test standard]
+require "bundler/gem_tasks"

data/app/controllers/account/webhooks/outgoing/deliveries_controller.rb

@@ -0,0 +1,79 @@
+class Account::Webhooks::Outgoing::DeliveriesController < Account::ApplicationController
+  account_load_and_authorize_resource :delivery, through: :endpoint, through_association: :deliveries
+
+  # GET /account/webhooks/outgoing/endpoints/:endpoint_id/deliveries
+  # GET /account/webhooks/outgoing/endpoints/:endpoint_id/deliveries.json
+  def index
+    # if you only want these objects shown on their parent's show page, uncomment this:
+    redirect_to [:account, @endpoint]
+  end
+
+  # GET /account/webhooks/outgoing/deliveries/:id
+  # GET /account/webhooks/outgoing/deliveries/:id.json
+  def show
+  end
+
+  # GET /account/webhooks/outgoing/endpoints/:endpoint_id/deliveries/new
+  def new
+  end
+
+  # GET /account/webhooks/outgoing/deliveries/:id/edit
+  def edit
+  end
+
+  # POST /account/webhooks/outgoing/endpoints/:endpoint_id/deliveries
+  # POST /account/webhooks/outgoing/endpoints/:endpoint_id/deliveries.json
+  def create
+    respond_to do |format|
+      if @delivery.save
+        format.html { redirect_to [:account, @endpoint, :deliveries], notice: I18n.t("webhooks/outgoing/deliveries.notifications.created") }
+        format.json { render :show, status: :created, location: [:account, @delivery] }
+      else
+        format.html { render :new, status: :unprocessable_entity }
+        format.json { render json: @delivery.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # PATCH/PUT /account/webhooks/outgoing/deliveries/:id
+  # PATCH/PUT /account/webhooks/outgoing/deliveries/:id.json
+  def update
+    respond_to do |format|
+      if @delivery.update(delivery_params)
+        format.html { redirect_to [:account, @delivery], notice: I18n.t("webhooks/outgoing/deliveries.notifications.updated") }
+        format.json { render :show, status: :ok, location: [:account, @delivery] }
+      else
+        format.html { render :edit, status: :unprocessable_entity }
+        format.json { render json: @delivery.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # DELETE /account/webhooks/outgoing/deliveries/:id
+  # DELETE /account/webhooks/outgoing/deliveries/:id.json
+  def destroy
+    @delivery.destroy
+    respond_to do |format|
+      format.html { redirect_to [:account, @endpoint, :deliveries], notice: I18n.t("webhooks/outgoing/deliveries.notifications.destroyed") }
+      format.json { head :no_content }
+    end
+  end
+
+  private
+
+  # Never trust parameters from the scary internet, only allow the white list through.
+  def delivery_params
+    strong_params = params.require(:webhooks_outgoing_delivery).permit(
+      :event_id,
+      :endpoint_url,
+      :delivered_at,
+      # 🚅 super scaffolding will insert new fields above this line.
+      # 🚅 super scaffolding will insert new arrays above this line.
+    )
+
+    assign_date_and_time(strong_params, :delivered_at)
+    # 🚅 super scaffolding will insert processing for new fields above this line.
+
+    strong_params
+  end
+end

data/app/controllers/account/webhooks/outgoing/delivery_attempts_controller.rb

@@ -0,0 +1,80 @@
+class Account::Webhooks::Outgoing::DeliveryAttemptsController < Account::ApplicationController
+  account_load_and_authorize_resource :delivery_attempt, through: :delivery, through_association: :delivery_attempts
+
+  # GET /account/webhooks/outgoing/deliveries/:delivery_id/delivery_attempts
+  # GET /account/webhooks/outgoing/deliveries/:delivery_id/delivery_attempts.json
+  def index
+    # if you only want these objects shown on their parent's show page, uncomment this:
+    redirect_to [:account, @delivery]
+  end
+
+  # GET /account/webhooks/outgoing/delivery_attempts/:id
+  # GET /account/webhooks/outgoing/delivery_attempts/:id.json
+  def show
+  end
+
+  # GET /account/webhooks/outgoing/deliveries/:delivery_id/delivery_attempts/new
+  def new
+  end
+
+  # GET /account/webhooks/outgoing/delivery_attempts/:id/edit
+  def edit
+  end
+
+  # POST /account/webhooks/outgoing/deliveries/:delivery_id/delivery_attempts
+  # POST /account/webhooks/outgoing/deliveries/:delivery_id/delivery_attempts.json
+  def create
+    respond_to do |format|
+      if @delivery_attempt.save
+        format.html { redirect_to [:account, @delivery, :delivery_attempts], notice: I18n.t("webhooks/outgoing/delivery_attempts.notifications.created") }
+        format.json { render :show, status: :created, location: [:account, @delivery_attempt] }
+      else
+        format.html { render :new, status: :unprocessable_entity }
+        format.json { render json: @delivery_attempt.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # PATCH/PUT /account/webhooks/outgoing/delivery_attempts/:id
+  # PATCH/PUT /account/webhooks/outgoing/delivery_attempts/:id.json
+  def update
+    respond_to do |format|
+      if @delivery_attempt.update(delivery_attempt_params)
+        format.html { redirect_to [:account, @delivery_attempt], notice: I18n.t("webhooks/outgoing/delivery_attempts.notifications.updated") }
+        format.json { render :show, status: :ok, location: [:account, @delivery_attempt] }
+      else
+        format.html { render :edit, status: :unprocessable_entity }
+        format.json { render json: @delivery_attempt.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # DELETE /account/webhooks/outgoing/delivery_attempts/:id
+  # DELETE /account/webhooks/outgoing/delivery_attempts/:id.json
+  def destroy
+    @delivery_attempt.destroy
+    respond_to do |format|
+      format.html { redirect_to [:account, @delivery, :delivery_attempts], notice: I18n.t("webhooks/outgoing/delivery_attempts.notifications.destroyed") }
+      format.json { head :no_content }
+    end
+  end
+
+  private
+
+  # Never trust parameters from the scary internet, only allow the white list through.
+  def delivery_attempt_params
+    strong_params = params.require(:webhooks_outgoing_delivery_attempt).permit(
+      :response_code,
+      :response_body,
+      :response_message,
+      :error_message,
+      :attempt_number,
+      # 🚅 super scaffolding will insert new fields above this line.
+      # 🚅 super scaffolding will insert new arrays above this line.
+    )
+
+    # 🚅 super scaffolding will insert processing for new fields above this line.
+
+    strong_params
+  end
+end

data/app/controllers/account/webhooks/outgoing/endpoints_controller.rb

@@ -0,0 +1,79 @@
+class Account::Webhooks::Outgoing::EndpointsController < Account::ApplicationController
+  account_load_and_authorize_resource :endpoint, through: :team, through_association: :webhooks_outgoing_endpoints
+
+  # GET /account/teams/:team_id/webhooks/outgoing/endpoints
+  # GET /account/teams/:team_id/webhooks/outgoing/endpoints.json
+  def index
+    # if you only want these objects shown on their parent's show page, uncomment this:
+    # redirect_to [:account, @team]
+  end
+
+  # GET /account/webhooks/outgoing/endpoints/:id
+  # GET /account/webhooks/outgoing/endpoints/:id.json
+  def show
+  end
+
+  # GET /account/teams/:team_id/webhooks/outgoing/endpoints/new
+  def new
+  end
+
+  # GET /account/webhooks/outgoing/endpoints/:id/edit
+  def edit
+  end
+
+  # POST /account/teams/:team_id/webhooks/outgoing/endpoints
+  # POST /account/teams/:team_id/webhooks/outgoing/endpoints.json
+  def create
+    respond_to do |format|
+      if @endpoint.save
+        format.html { redirect_to [:account, @team, :webhooks_outgoing_endpoints], notice: I18n.t("webhooks/outgoing/endpoints.notifications.created") }
+        format.json { render :show, status: :created, location: [:account, @endpoint] }
+      else
+        format.html { render :new, status: :unprocessable_entity }
+        format.json { render json: @endpoint.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # PATCH/PUT /account/webhooks/outgoing/endpoints/:id
+  # PATCH/PUT /account/webhooks/outgoing/endpoints/:id.json
+  def update
+    respond_to do |format|
+      if @endpoint.update(endpoint_params)
+        format.html { redirect_to [:account, @endpoint], notice: I18n.t("webhooks/outgoing/endpoints.notifications.updated") }
+        format.json { render :show, status: :ok, location: [:account, @endpoint] }
+      else
+        format.html { render :edit, status: :unprocessable_entity }
+        format.json { render json: @endpoint.errors, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  # DELETE /account/webhooks/outgoing/endpoints/:id
+  # DELETE /account/webhooks/outgoing/endpoints/:id.json
+  def destroy
+    @endpoint.destroy
+    respond_to do |format|
+      format.html { redirect_to [:account, @team, :webhooks_outgoing_endpoints], notice: I18n.t("webhooks/outgoing/endpoints.notifications.destroyed") }
+      format.json { head :no_content }
+    end
+  end
+
+  private
+
+  # Never trust parameters from the scary internet, only allow the white list through.
+  def endpoint_params
+    strong_params = params.require(:webhooks_outgoing_endpoint).permit(
+      :name,
+      :url,
+      # 🚅 super scaffolding will insert new fields above this line.
+      event_type_ids: [],
+      # 🚅 super scaffolding will insert new arrays above this line.
+    )
+
+    assign_select_options(strong_params, :event_type_ids)
+    # 🚅 super scaffolding will insert processing for new fields above this line.
+
+    strong_params
+  end
+end

data/app/controllers/api/v1/webhooks/outgoing/deliveries_endpoint.rb

@@ -0,0 +1,63 @@
+class Api::V1::Webhooks::Outgoing::DeliveriesEndpoint < Api::V1::Root
+  helpers do
+    params :endpoint_id do
+      requires :endpoint_id, type: Integer, allow_blank: false, desc: "Endpoint ID"
+    end
+
+    params :id do
+      requires :id, type: Integer, allow_blank: false, desc: "Delivery ID"
+    end
+
+    params :delivery do
+      optional :event_id, type: String, desc: Api.heading(:event_id)
+      optional :endpoint_url, type: String, desc: Api.heading(:endpoint_url)
+      optional :delivered_at, type: DateTime, desc: Api.heading(:delivered_at)
+      # 🚅 super scaffolding will insert new fields above this line.
+      # 🚅 super scaffolding will insert new arrays above this line.
+
+      # 🚅 super scaffolding will insert processing for new fields above this line.
+    end
+  end
+
+  resource "webhooks/outgoing/endpoints", desc: Api.title(:collection_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::Delivery
+    end
+
+    #
+    # INDEX
+    #
+
+    desc Api.title(:index), &Api.index_desc
+    params do
+      use :endpoint_id
+    end
+    oauth2
+    paginate per_page: 100
+    get "/:endpoint_id/deliveries" do
+      @paginated_deliveries = paginate @deliveries
+      render @paginated_deliveries, serializer: Api.serializer
+    end
+  end
+
+  resource "webhooks/outgoing/deliveries", desc: Api.title(:member_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::Delivery
+    end
+
+    #
+    # SHOW
+    #
+
+    desc Api.title(:show), &Api.show_desc
+    params do
+      use :id
+    end
+    oauth2
+    route_param :id do
+      get do
+        render @delivery, serializer: Api.serializer
+      end
+    end
+  end
+end

data/app/controllers/api/v1/webhooks/outgoing/delivery_attempts_endpoint.rb

@@ -0,0 +1,65 @@
+class Api::V1::Webhooks::Outgoing::DeliveryAttemptsEndpoint < Api::V1::Root
+  helpers do
+    params :delivery_id do
+      requires :delivery_id, type: Integer, allow_blank: false, desc: "Delivery ID"
+    end
+
+    params :id do
+      requires :id, type: Integer, allow_blank: false, desc: "Delivery Attempt ID"
+    end
+
+    params :delivery_attempt do
+      optional :response_code, type: String, desc: Api.heading(:response_code)
+      optional :response_body, type: String, desc: Api.heading(:response_body)
+      optional :response_message, type: String, desc: Api.heading(:response_message)
+      optional :error_message, type: String, desc: Api.heading(:error_message)
+      optional :attempt_number, type: String, desc: Api.heading(:attempt_number)
+      # 🚅 super scaffolding will insert new fields above this line.
+      # 🚅 super scaffolding will insert new arrays above this line.
+
+      # 🚅 super scaffolding will insert processing for new fields above this line.
+    end
+  end
+
+  resource "webhooks/outgoing/deliveries", desc: Api.title(:collection_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::DeliveryAttempt
+    end
+
+    #
+    # INDEX
+    #
+
+    desc Api.title(:index), &Api.index_desc
+    params do
+      use :delivery_id
+    end
+    oauth2
+    paginate per_page: 100
+    get "/:delivery_id/delivery_attempts" do
+      @paginated_delivery_attempts = paginate @delivery_attempts
+      render @paginated_delivery_attempts, serializer: Api.serializer
+    end
+  end
+
+  resource "webhooks/outgoing/delivery_attempts", desc: Api.title(:member_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::DeliveryAttempt
+    end
+
+    #
+    # SHOW
+    #
+
+    desc Api.title(:show), &Api.show_desc
+    params do
+      use :id
+    end
+    oauth2
+    route_param :id do
+      get do
+        render @delivery_attempt, serializer: Api.serializer
+      end
+    end
+  end
+end

data/app/controllers/api/v1/webhooks/outgoing/endpoints_endpoint.rb

@@ -0,0 +1,119 @@
+class Api::V1::Webhooks::Outgoing::EndpointsEndpoint < Api::V1::Root
+  helpers do
+    params :team_id do
+      requires :team_id, type: Integer, allow_blank: false, desc: "Team ID"
+    end
+
+    params :id do
+      requires :id, type: Integer, allow_blank: false, desc: "Endpoint ID"
+    end
+
+    params :endpoint do
+      optional :name, type: String, desc: Api.heading(:name)
+      optional :url, type: String, desc: Api.heading(:url)
+      # 🚅 super scaffolding will insert new fields above this line.
+      optional :event_type_ids, type: Array, desc: Api.heading(:event_type_ids)
+      # 🚅 super scaffolding will insert new arrays above this line.
+
+      # 🚅 super scaffolding will insert processing for new fields above this line.
+    end
+  end
+
+  resource "teams", desc: Api.title(:collection_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::Endpoint
+    end
+
+    #
+    # INDEX
+    #
+
+    desc Api.title(:index), &Api.index_desc
+    params do
+      use :team_id
+    end
+    oauth2
+    paginate per_page: 100
+    get "/:team_id/webhooks/outgoing/endpoints" do
+      @paginated_endpoints = paginate @endpoints
+      render @paginated_endpoints, serializer: Api.serializer
+    end
+
+    #
+    # CREATE
+    #
+
+    desc Api.title(:create), &Api.create_desc
+    params do
+      use :team_id
+      use :endpoint
+    end
+    route_setting :api_resource_options, permission: :create
+    oauth2 "write"
+    post "/:team_id/webhooks/outgoing/endpoints" do
+      if @endpoint.save
+        render @endpoint, serializer: Api.serializer
+      else
+        record_not_saved @endpoint
+      end
+    end
+  end
+
+  resource "webhooks/outgoing/endpoints", desc: Api.title(:member_actions) do
+    after_validation do
+      load_and_authorize_api_resource Webhooks::Outgoing::Endpoint
+    end
+
+    #
+    # SHOW
+    #
+
+    desc Api.title(:show), &Api.show_desc
+    params do
+      use :id
+    end
+    oauth2
+    route_param :id do
+      get do
+        render @endpoint, serializer: Api.serializer
+      end
+    end
+
+    #
+    # UPDATE
+    #
+
+    desc Api.title(:update), &Api.update_desc
+    params do
+      use :id
+      use :endpoint
+    end
+    route_setting :api_resource_options, permission: :update
+    oauth2 "write"
+    route_param :id do
+      put do
+        if @endpoint.update(declared(params, include_missing: false))
+          render @endpoint, serializer: Api.serializer
+        else
+          record_not_saved @endpoint
+        end
+      end
+    end
+
+    #
+    # DESTROY
+    #
+
+    desc Api.title(:destroy), &Api.destroy_desc
+    params do
+      use :id
+    end
+    route_setting :api_resource_options, permission: :destroy
+    oauth2 "delete"
+    route_param :id do
+      delete do
+        render @endpoint.destroy, serializer: Api.serializer
+      end
+    end
+  end
+end

data/app/jobs/webhooks/outgoing/delivery_job.rb

@@ -0,0 +1,7 @@
+class Webhooks::Outgoing::DeliveryJob < ApplicationJob
+  queue_as :default
+
+  def perform(delivery)
+    delivery.deliver
+  end
+end

data/{lib → app}/models/concerns/webhooks/outgoing/issuing_model.rb

@@ -13,29 +13,21 @@ module Webhooks::Outgoing::IssuingModel
   module ClassMethods
   end
 
-  def webhooks_team
-    send(team_association)
-  end
-
-  def team_association
-    Webhooks::Outgoing.team_model.underscore.to_sym
-  end
-
   def generate_webhook(action)
     # we can only generate webhooks for objects that return their team.
-    return unless respond_to? team_association
+    return unless respond_to? :team
 
     # Try to find an event type definition for this action.
     event_type = Webhooks::Outgoing::EventType.find_by(id: "#{self.class.name.underscore}.#{action}")
 
     # If the event type is defined as one that people can be subscribed to,
     # and this object has a team where an associated outgoing webhooks endpoint could be registered.
-    if event_type && webhooks_team
+    if event_type && team
 
       # Only generate an event record if an endpoint is actually listening for this event type.
-      if webhooks_team.webhooks_outgoing_endpoints.listening_for_event_type_id(event_type.id).any?
+      if team.webhooks_outgoing_endpoints.listening_for_event_type_id(event_type.id).any?
         data = "Api::V1::#{self.class.name}Serializer".constantize.new(self).serializable_hash[:data]
-        webhook = webhooks_team.webhooks_outgoing_events.create(event_type_id: event_type.id, subject: self, data: data)
+        webhook = team.webhooks_outgoing_events.create(event_type_id: event_type.id, subject: self, data: data)
         webhook.deliver
       end
     end
@@ -50,8 +42,8 @@ module Webhooks::Outgoing::IssuingModel
   end
 
   def generate_deleted_webhook
-    return false unless respond_to?(team_association)
-    return false if webhooks_team&.being_destroyed?
+    return false unless respond_to?(:team)
+    return false if team&.being_destroyed?
     generate_webhook("deleted")
   end
 end