shopify-gold 4.2.0 → 5.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad7514ffdfb66670043eac39bffbd3f1db861f45c6befd38bb92e570bcd4fd95
-  data.tar.gz: f03dc8cad6c26749cd70d8357e2019695102d944ddfb4300ce33e55937c5d224
+  metadata.gz: d15786e3bd37ff9291bf4f000c3f580373a56dcf5bd8662df743ec4a30ec5c36
+  data.tar.gz: 9f838172385fffd62d0397c92db4acb8a8bda60c7aac144bfdaf90883d6ba452
 SHA512:
-  metadata.gz: bddd6866e01bc82889115697f3cd3eb5cedf472942640e7c13ed3cc9678feac878179fbd20f027854de47d471dfa65bd548fd8fbb9d489cbede3bb63c81b77f4
-  data.tar.gz: 3db49c76d9554f994852112ce046040b8a7c2ee09b43c5588d75209b5361ac029e7372cb8db431c59f6fab4e5b9dde49d4449fb51dd0798c7fcecf2297361b3b
+  metadata.gz: 9eb7a98aab47e7948e56b3522586ca3023bbf19e9fd3df2c7cac5036bf8986b8abfc963ec9173056999aef245a0879d0cdc7f0f5dc0b62c5145131f97571c556
+  data.tar.gz: 3c642760a522e4e1c8cac183f6eb2b7f5e36d8bb2d885bb34de7faf83cd0c237ee6d93d6771328efe742a5619cb44178912529a4b9287fd72d383be442f896b7

data/app/controllers/gold/authenticated_controller.rb

@@ -1,4 +1,4 @@
-require_dependency "gold/application_controller"
+require "gold/application_controller"
 
 module Gold
   # Inherit from Shopify's authenticated module

data/app/controllers/gold/billing_controller.rb

@@ -7,7 +7,7 @@ module Gold
     layout "gold/billing"
 
     rescue_from Statesman::TransitionFailedError do |e|
-      billing = Billing.find_by!(shop_id: session[:shopify])
+      billing = Billing.find_by!(shop_id: session[:shop_id])
       Gold.logger.error("Shop '#{billing.shop.shopify_domain}' failed to " \
                         "transtion '#{e}'")
       render "gold/billing/transition_error", layout: "gold/billing",
@@ -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

@@ -9,7 +9,7 @@ module Gold
 
       # Returns the Gold::Billing instance for the currently logged-in merchant.
       def billing
-        @billing ||= Billing.find_by!(shop_id: session[:shopify])
+        @billing ||= Billing.find_by!(shop_id: session[:shop_id])
       rescue ActiveRecord::RecordNotFound
         raise Gold::Exceptions::BillingNotFound
       end

data/app/models/gold/concerns/gilded.rb

@@ -1,4 +1,4 @@
-require_dependency "statesman"
+require "statesman"
 
 module Gold
   module Concerns

data/app/models/gold/machine.rb

@@ -1,4 +1,4 @@
-require_dependency "statesman"
+require "statesman"
 
 module Gold
   # This is the finite state machine specification that governs how shops may
@@ -230,7 +230,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 +239,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 +258,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/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 = "4.2.0".freeze
+  VERSION = "5.2.1".freeze
 end

data/lib/tasks/gold_tasks.rake

@@ -66,6 +66,9 @@ namespace :gold do
         Gold::UninstallOp.new(billing).call
       rescue ActiveResource::ClientError => e
         puts "Error for '#{billing.shop.shopify_domain}', #{e}"
+      rescue Statesman::GuardFailedError,
+             Statesman::TransitionFailedError => e
+        puts "Cannot transition: #{e.message}"
       end
     end
   end
@@ -90,8 +93,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: 4.2.0
+  version: 5.2.1
 platform: ruby
 authors:
 - Andrew Smith
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-07 00:00:00.000000000 Z
+date: 2020-07-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -17,42 +17,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: '6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: '6'
 - !ruby/object:Gem::Dependency
   name: sass-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: shopify_app
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11'
+        version: '13'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11'
+        version: '13'
 - !ruby/object:Gem::Dependency
   name: statesman
   requirement: !ruby/object:Gem::Requirement
@@ -73,14 +73,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.5'
+        version: '2.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.5'
+        version: '2.7'
 - !ruby/object:Gem::Dependency
   name: faker
   requirement: !ruby/object:Gem::Requirement
@@ -113,16 +113,16 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.74.0
+        version: '0.74'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.74.0
+        version: '0.74'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
@@ -157,14 +157,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 11.3.0
+        version: '13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 11.3.0
+        version: '13'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -185,14 +185,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.2
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.2
+        version: '3'
 description: |
   Gold is Helium's approach to billing for Shopify apps. It provides a
   framework to build Rails apps upon.
@@ -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