bullet_train-integrations-stripe 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 601ccfba8cfe6ad2c8291a15d288076eae7936ac68184672cac9bc141652f181
-  data.tar.gz: 3b2e87456e3693a997a93f549bbb834341a4d8eabaf21367b5ec2d4e75ca80d8
+  metadata.gz: b6725113b0b72922229fe124a7832ee8eaa869eaed3952215915dabf0c1fd0e5
+  data.tar.gz: 5e576403715a63ad350964004995caba8fb74fdefe4dac40f10cd0fea9baed4c
 SHA512:
-  metadata.gz: b7098328420c44ed0eedd22dc400fdc29663c149d9b59131965e397a8f4cb27efba901b0e6f7433bce515abb2a8f9aa596e4b984ebdecd6418ed26b03f9ffa01
-  data.tar.gz: ea5411f17822836316a664c21db3ff6e170fb96747536a42617c759909d8e299762721796caa1fc233670effd5e1000f213784f406d6e1d2217728d52e4f749a
+  metadata.gz: 3d215c921514dba70f674ce3afda981e6ecc084b817d08f8d0d400f63d8d34fe93e134d44d6ae533fde2dcd53f2cb6f33c5ebd8fa2b74362f9d645acc94f815b
+  data.tar.gz: 9508e0f4ffd9a2d0d9635b41d96d9ed263640aaec071e1337743fad4178d485490a87d5729551e32f86a04fe71e56ce3026fdcfd8a2046e13dc23611576c7e30

data/app/models/concerns/integrations/stripe_installations/base.rb

@@ -0,0 +1,25 @@
+module Integrations::StripeInstallations::Base
+  extend ActiveSupport::Concern
+  # 🚅 add concerns above.
+
+  included do
+    belongs_to :team
+    belongs_to :oauth_stripe_account, class_name: "Oauth::StripeAccount"
+    # 🚅 add belongs_to associations above.
+
+    # 🚅 add has_many associations above.
+
+    # 🚅 add has_one associations above.
+
+    # 🚅 add scopes above.
+
+    validates :name, presence: true
+    # 🚅 add validations above.
+
+    # 🚅 add callbacks above.
+
+    # 🚅 add delegations above.
+  end
+
+  # 🚅 add methods above.
+end

data/app/models/concerns/oauth/stripe_accounts/base.rb

@@ -0,0 +1,48 @@
+module Oauth::StripeAccounts::Base
+  extend ActiveSupport::Concern
+  # 🚅 add concerns above.
+
+  included do
+    belongs_to :user, optional: true
+    # 🚅 add belongs_to associations above.
+
+    has_many :webhooks_incoming_oauth_stripe_account_webhooks, class_name: "Webhooks::Incoming::Oauth::StripeAccountWebhook", foreign_key: "oauth_stripe_account_id"
+    has_many :integrations_stripe_installations, class_name: "Integrations::StripeInstallation", foreign_key: "oauth_stripe_account_id"
+    # 🚅 add has_many associations above.
+
+    # 🚅 add has_one associations above.
+
+    # 🚅 add scopes above.
+
+    validates :uid, presence: true
+    # 🚅 add validations above.
+
+    # 🚅 add callbacks above.
+
+    # 🚅 add delegations above.
+  end
+
+  def label_string
+    name
+  end
+
+  # TODO You should update this with an implementation appropriate for the provider you're integrating with.
+  # This must return _something_, otherwise new installations won't save.
+  def name
+    data.dig("info", "name").presence || "Stripe Account"
+  rescue
+    "Stripe Account"
+  end
+
+  def name_was
+    name
+  end
+
+  def update_from_oauth(auth)
+    self.uid = auth.uid
+    self.data = auth
+    save
+  end
+
+  # 🚅 add methods above.
+end

data/app/models/concerns/webhooks/incoming/oauth/stripe_account_webhooks/base.rb

@@ -0,0 +1,37 @@
+module Webhooks::Incoming::Oauth::StripeAccountWebhooks::Base
+  extend ActiveSupport::Concern
+
+  included do
+    include Webhooks::Incoming::Webhook
+
+    belongs_to :oauth_stripe_account, class_name: "Oauth::StripeAccount", optional: true
+  end
+
+  def process
+    # if this is a stripe connect webhook ..
+    if data["account"]
+
+      # if we're able to find an account in our system that this webhook should be routed to ..
+      if (self.oauth_stripe_account = Oauth::StripeAccount.find_by(uid: data["account"]))
+
+        # save the reference to the account.
+        save
+
+        # delegate processing to that account.
+        oauth_stripe_account.process_webhook(self)
+
+      end
+
+      # if we didn't find an account for the webhook, they've probably deleted their account. we'll just ignore it for
+      # now, but it's still in our database for debugging purposes. we'll probably want to send a request to stripe
+      # in order to disconnect their account from our application so we stop receiving webhooks.
+
+    else
+
+      # it's possible we're receiving a general webhook that isn't specific to an account.
+      # however, we want to know about these, so raise an error.
+      raise "received a webhook we weren't expecting"
+
+    end
+  end
+end

data/app/models/integrations/stripe_installation.rb

@@ -1,29 +1,7 @@
 class Integrations::StripeInstallation < ApplicationRecord
-  # 🚅 add concerns above.
-
-  belongs_to :team
-  belongs_to :oauth_stripe_account, class_name: "Oauth::StripeAccount"
-  # 🚅 add belongs_to associations above.
-
-  # 🚅 add has_many associations above.
-
-  # 🚅 add has_one associations above.
-
-  # 🚅 add scopes above.
-
-  validates :name, presence: true
-  # 🚅 add validations above.
-
-  # 🚅 add callbacks above.
-
-  # 🚅 add delegations above.
+  include Integrations::StripeInstallations::Base
 
   def process_webhook(webhook)
-    # consider using transactions here. if an error occurs in the processing of a webhook, it's not like user-facing
-    # errors on the web where they see a red screen of death. instead, sidekiq will reattempt the processing of the
-    # entire webhook, which means that earlier portions of your logic will be run more than once unless you're careful
-    # to avoid it.
+    raise "You should implement a `Integrations::StripeInstallation` model in your application that has `include Integrations::StripeInstallations::Base` and implements a `def process_webhook(webhook)` method."
   end
-
-  # 🚅 add methods above.
 end

data/app/models/oauth/stripe_account.rb

@@ -1,53 +1,3 @@
 class Oauth::StripeAccount < ApplicationRecord
-  # 🚅 add concerns above.
-
-  belongs_to :user, optional: true
-  # 🚅 add belongs_to associations above.
-
-  has_many :webhooks_incoming_oauth_stripe_account_webhooks, class_name: "Webhooks::Incoming::Oauth::StripeAccountWebhook", foreign_key: "oauth_stripe_account_id"
-  has_many :integrations_stripe_installations, class_name: "Integrations::StripeInstallation", foreign_key: "oauth_stripe_account_id"
-  # 🚅 add has_many associations above.
-
-  # 🚅 add has_one associations above.
-
-  # 🚅 add scopes above.
-
-  validates :uid, presence: true
-  # 🚅 add validations above.
-
-  # 🚅 add callbacks above.
-
-  # 🚅 add delegations above.
-
-  def label_string
-    name
-  end
-
-  # TODO You should update this with an implementation appropriate for the provider you're integrating with.
-  # This must return _something_, otherwise new installations won't save.
-  def name
-    data.dig("info", "name").presence || "Stripe Account"
-  rescue
-    "Stripe Account"
-  end
-
-  def name_was
-    name
-  end
-
-  def update_from_oauth(auth)
-    self.uid = auth.uid
-    self.data = auth
-    save
-  end
-
-  # webhooks received for this account will be routed here asynchronously for processing on a worker.
-  def process_webhook(webhook)
-    # we delegate processing to any active installations.
-    integrations_stripe_installations.order(:id).each do |installation|
-      installation.process_webhook(webhook)
-    end
-  end
-
-  # 🚅 add methods above.
+  include Oauth::StripeAccounts::Base
 end

data/app/models/webhooks/incoming/oauth/stripe_account_webhook.rb

@@ -1,33 +1,3 @@
 class Webhooks::Incoming::Oauth::StripeAccountWebhook < ApplicationRecord
-  include Webhooks::Incoming::Webhook
-
-  belongs_to :oauth_stripe_account, class_name: "Oauth::StripeAccount", optional: true
-
-  def process
-    # if this is a stripe connect webhook ..
-    if data["account"]
-
-      # if we're able to find an account in our system that this webhook should be routed to ..
-      if (self.oauth_stripe_account = Oauth::StripeAccount.find_by(uid: data["account"]))
-
-        # save the reference to the account.
-        save
-
-        # delegate processing to that account.
-        oauth_stripe_account.process_webhook(self)
-
-      end
-
-      # if we didn't find an account for the webhook, they've probably deleted their account. we'll just ignore it for
-      # now, but it's still in our database for debugging purposes. we'll probably want to send a request to stripe
-      # in order to disconnect their account from our application so we stop receiving webhooks.
-
-    else
-
-      # it's possible we're receiving a general webhook that isn't specific to an account.
-      # however, we want to know about these, so raise an error.
-      raise "received a webhook we weren't expecting"
-
-    end
-  end
+  include Webhooks::Incoming::Oauth::StripeAccountWebhooks::Base
 end

data/lib/bullet_train/integrations/stripe/version.rb

@@ -1,7 +1,7 @@
 module BulletTrain
   module Integrations
     module Stripe
-      VERSION = "1.0.0"
+      VERSION = "1.0.1"
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-integrations-stripe
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Andrew Culver
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-22 00:00:00.000000000 Z
+date: 2022-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -38,6 +38,9 @@ files:
 - app/controllers/account/integrations/stripe_installations_controller.rb
 - app/controllers/account/oauth/stripe_accounts_controller.rb
 - app/controllers/webhooks/incoming/oauth/stripe_account_webhooks_controller.rb
+- app/models/concerns/integrations/stripe_installations/base.rb
+- app/models/concerns/oauth/stripe_accounts/base.rb
+- app/models/concerns/webhooks/incoming/oauth/stripe_account_webhooks/base.rb
 - app/models/integrations/stripe_installation.rb
 - app/models/oauth/stripe_account.rb
 - app/models/webhooks/incoming/oauth/stripe_account_webhook.rb