shopify-gold 5.0.0 → 5.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59f64edfc44f572a2fdbf18a0c7d4e87f282ec34c496f670d25639f4ea680874
-  data.tar.gz: 4da2cdbd7b0438f483abc4aa04fa904a7f0be3f6afa93a5024505235ab1c34bc
+  metadata.gz: 626161bde9c84b77043dda68cd9c989f9a5b8d0e20238bc9cfef5762ea5812e1
+  data.tar.gz: 9e2180c3e8a4471c4ae9a3e29d52e6374b16a647dc2602f3e2e88add1df1e39b
 SHA512:
-  metadata.gz: 31917b72d24e6fc5c27e3bdaccb0246194344e57272f6da64e7d6cd361d1f57255920d9a8e33990cebb5c050ffb57f64a5806496e9b7ff27196fe8d946c41054
-  data.tar.gz: 1280caec87e5918c86a0f84d4162141ad4909812d2ce90cf28f7b03a066d702119e9b42287a5b96db95891166b44d049fe0d714d76663b8dc1a02ea31c489c06
+  metadata.gz: 69377f5547134f2e571ba9a4dadf7cdef990489bfae31f998548ec8cc701078f907719fc7c66dcc07297c79910bb44a4c3c875d8e22b1a8db66585322dbe69dc
+  data.tar.gz: aca74451cd78c8543067fadcf69cc9c7473a16ca2920f1880b94129f421f31c98f1bf08a270cc1fd12379cbc47ebdbe56e7b72aefdc35516e77695f0f8e185fb

data/app/controllers/gold/billing_controller.rb

@@ -57,7 +57,7 @@ module Gold
       case outcome
       when SameTier, TierApplied
         cookies.delete(:gold_tier_id)
-        redirect_to main_app.root_url
+        redirect_to Gold.configuration.after_tier_redirect_path
       when CannotSelectTier
         flash.now[:error] = "Your shop is not eligible for this plan"
         render :tier
@@ -98,7 +98,7 @@ module Gold
           Gold.logger.info("[#{billing.id}] Charge is ready")
         end
 
-        redirect_to main_app.root_url
+        redirect_to Gold.configuration.after_tier_redirect_path
       else
         raise "Not sure how to handle #{outcome} on outstanding charge"
       end
@@ -112,7 +112,7 @@ module Gold
       case outcome
       when ActiveCharge
         ApplyTierOp.new(billing).call
-        outer_redirect_to_main
+        outer_redirect(Gold.configuration.after_tier_redirect_path)
       when DeclinedCharge
         redirect_to declined_charge_url
       when ExpiredCharge
@@ -165,14 +165,14 @@ module Gold
 
     # If embedded app, redirect directly through the Shopify admin. This can provide
     # a better experience than the page rendering and JS kicking off a redirect
-    def outer_redirect_to_main
+    def outer_redirect(redirect_path)
       if ShopifyApp.configuration.embedded_app
         shopify_domain = billing.shop.shopify_domain
         api_key = ShopifyApp.configuration.api_key
-        fullpath = "/admin/apps/#{api_key}#{main_app.root_path}"
+        fullpath = "/admin/apps/#{api_key}#{redirect_path}"
         redirect_to URI::HTTPS.build(host: shopify_domain, path: fullpath).to_s
       else
-        redirect_to main_app.root_url
+        redirect_to redirect_path
       end
     end
 

data/app/controllers/gold/concerns/merchant_facing.rb

@@ -49,7 +49,7 @@ module Gold
           Gold.logger.info("Confronting billing, state is '#{state}'")
 
           case state
-          when nil, :new
+          when nil, :new, :install
             Gold.logger.info("Redirecting to terms page...")
             return @context.redirect_to(@engine.terms_url)
           when :select_tier, :reinstalled, :accepted_terms

data/app/models/gold/machine.rb

@@ -12,6 +12,9 @@ module Gold
 
     state :new, initial: true
 
+    # A merchant has installed the app
+    state :install
+
     # Before merchants can use our app, they will need to review and accept our
     # Terms of Service.
     state :accepted_terms
@@ -165,7 +168,8 @@ module Gold
 
     # Transitions
 
-    transition from: :new, to: %i[accepted_terms marked_as_uninstalled]
+    transition from: :new, to: %i[install marked_as_uninstalled]
+    transition from: :install, to: %i[accepted_terms marked_as_uninstalled]
     transition from: :accepted_terms, to: %i[select_tier marked_as_uninstalled]
 
     before_transition to: :select_tier, &require_metadata(:tier_id)
@@ -230,7 +234,8 @@ module Gold
                                           apply_free_tier
                                           change_tier
                                           staff
-                                          optional_charge]
+                                          optional_charge
+                                          marked_as_uninstalled]
 
     before_transition to: :optional_charge, &require_metadata(:charge_id)
     guard_transition to: :optional_charge, &ensure_plan_is(:paying?)
@@ -238,7 +243,8 @@ module Gold
                                               optional_charge_accepted
                                               optional_charge_declined
                                               marked_as_uninstalled]
-    transition from: :optional_charge_declined, to: :billing
+    transition from: :optional_charge_declined, to: %i[billing
+                                                       marked_as_uninstalled]
 
     transition from: :optional_charge_accepted, to: %i[charge_activated
                                                        marked_as_uninstalled]
@@ -256,13 +262,17 @@ module Gold
                                                       marked_as_uninstalled]
 
     transition from: :delayed_charge_expired, to: %i[delayed_charge
-                                                     marked_as_delinquent]
+                                                     marked_as_delinquent
+                                                     marked_as_uninstalled]
     transition from: :delayed_charge_declined, to: %i[marked_as_delinquent
-                                                      delayed_charge]
+                                                      delayed_charge
+                                                      marked_as_uninstalled]
 
     guard_transition to: :marked_as_delinquent,
                      &ensure_min_days_in_state(Gold.configuration.days_until_delinquent)
-    transition from: :marked_as_delinquent, to: :delinquent
+    transition from: :marked_as_delinquent, to: %i[delinquent
+                                                   marked_as_uninstalled]
+
     transition from: :delinquent, to: %i[marked_as_uninstalled delayed_charge cleanup]
 
     guard_transition to: :affiliate, &ensure_plan_is(:affiliate?)

data/app/models/gold/shopify_plan.rb

@@ -20,14 +20,16 @@ module Gold
 
     # Returns true if this is a development (non-live) shop.
     def affiliate?
-      plan == "affiliate"
+      plans = %w[affiliate partner_test]
+      plans.include?(plan)
     end
 
     # Returns true if this is a shop owned by Shopify staff. This specifically
     # excludes Shopify Business shops, as we believe those are paid stores that
     # Shopify employees use for their own businesses.
     def staff?
-      plan == "staff"
+      plans = %w[plus_partner_sandbox staff]
+      plans.include?(plan)
     end
 
     # Returns true if this shop has been frozen by Shopify for non-payment.

data/app/models/gold/tier.rb

@@ -79,11 +79,11 @@ module Gold
     end
 
     def qualifications
-      parent ? parent.qualifications : @qualifications
+      parent ? parent.qualifications.merge(@qualifications) : @qualifications
     end
 
     def features
-      parent ? parent.features : @features
+      parent ? parent.features.merge(@features) : @features
     end
 
     def parent

data/app/operations/gold/install_op.rb

@@ -10,11 +10,13 @@ module Gold
     end
 
     def call
-      # If possible, transition to reinstalled
+      # Transition to reinstalled, if possible
       @billing.transition_to(:reinstalled)
 
       case @billing.current_state
       when :new, :reinstalled
+        # Transition to install, if possible
+        @billing.transition_to(:install)
         Gold.configuration.on_install&.call(@billing)
       when :frozen
         return CheckChargeOp.new(@billing).call

data/lib/gold/billing_migrator.rb

@@ -45,6 +45,8 @@ module Gold
         trial_starts_at: @trial_starts_at
       )
 
+      @billing.transition_to! :install
+
       lookup_tier!
       accept_terms
       select_tier

data/lib/gold/configuration.rb

@@ -13,7 +13,8 @@ module Gold
                   :days_until_cleanup,
                   :shop_domain_attribute,
                   :force_embedded_redirect,
-                  :app_listing_url
+                  :app_listing_url,
+                  :after_tier_redirect_path
 
     # Callbacks
     attr_accessor :on_terms,
@@ -72,6 +73,10 @@ module Gold
       # The URL to follow after a referral code is tracked
       # (e.g. https://apps.shopify.com/customr)
       @app_listing_url = "https://apps.shopify.com/"
+
+      # The redirect path after a tier is applied. Use this path to welcome a
+      # merchant if they just installed
+      @after_tier_redirect_path = "/"
     end
 
     def shop_class

data/lib/gold/version.rb

@@ -1,3 +1,3 @@
 module Gold
-  VERSION = "5.0.0".freeze
+  VERSION = "5.4.0".freeze
 end

data/lib/tasks/gold_tasks.rake

@@ -66,6 +66,8 @@ namespace :gold do
         Gold::UninstallOp.new(billing).call
       rescue ActiveResource::ClientError => e
         puts "Error for '#{billing.shop.shopify_domain}', #{e}"
+      rescue Statesman::GuardFailedError => e
+        puts "Cannot transition: #{e.message}"
       end
     end
   end
@@ -90,8 +92,17 @@ namespace :gold do
       end
 
       billing.shop.with_shopify_session do
-        Gold::MarkAsDelinquentOp.new(billing).call
-        puts "Shop '#{billing.shop.shopify_domain}' is delinquent"
+        domain = billing.shop.shopify_domain
+
+        begin
+          Gold::MarkAsDelinquentOp.new(billing).call
+          puts "Shop '#{domain}' is delinquent"
+        rescue ActiveResource::UnauthorizedAccess
+          puts "It looks like '#{domain}' uninstalled, running op..."
+          Gold::UninstallOp.new(billing).call
+        rescue ActiveResource::ClientError => e
+          puts "Error for '#{domain}', #{e}"
+        end
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shopify-gold
 version: !ruby/object:Gem::Version
-  version: 5.0.0
+  version: 5.4.0
 platform: ruby
 authors:
 - Andrew Smith
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-09 00:00:00.000000000 Z
+date: 2021-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -45,14 +45,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '16'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '16'
 - !ruby/object:Gem::Dependency
   name: statesman
   requirement: !ruby/object:Gem::Requirement
@@ -157,14 +157,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '16'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -317,7 +317,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Helium's approach to billing for Shopify apps