solidus_backtracs 2.2.0 → 2.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7e943693a25b935abcf72d941f4f38c891761c3afae537f108e211041ccdc6e2
-  data.tar.gz: 87c9e1870799e0b007a8e2011a3460086b0aa9ccb39fa8a34bd1942bdf38e3ba
+  metadata.gz: ccacbc5731b470aa3dd924c0035d026948ecd49042199644ad5050e9793288be
+  data.tar.gz: dc1e92c7d93540ce5d7cd5a6ceba48cebc49d2f79f1da8a37dba76288a78dfca
 SHA512:
-  metadata.gz: c43355431c15e927961f74576d56e80bf7e56f6a14233bdf0ade847e833624c60cdf110c377952014e2007a0cb4d4138e8c5300e3e444a9d77f39c83effda44f
-  data.tar.gz: 95ce5f1c0d8d86a5a07eb884154caeecb9ca3c69a634594964d795aaa80439bb6a0bec00074dcf9151eeb16d02099fbea21a47fdcb560a9e98975e1f69892f1e
+  metadata.gz: f2a8a34c29deb725c01147894988c25c24783f6fd03e23a51d268abbf6b819cef0619889697f485989e870f52839a8047838cb9a7f159bfee369782f9e0e5bc7
+  data.tar.gz: c8d78430bc2311f1484e93c1244b83d6a80f4d4d63c5e91fe1bb7080912deea64d96c23789beba0bea9a9c1a185d2dabfc4647fec0524070d4baa71c553d41e1

data/.gitignore

@@ -18,3 +18,5 @@ spec/examples.txt
 .rvmrc
 .ruby-version
 .ruby-gemset
+.tool-versions
+TAGS

data/CHANGELOG.md

@@ -1,2 +1,16 @@
 # Changelog
 
+## 2.2.0
+
+Support the Solidus Assembly gem, consider allowed SKUs within assemblies/bundles
+Retry logic for the authenticated request
+
+## 2.1.0
+
+New auth and API patterns for Backtracs
+Support specific SKUs to be mapped
+
+## 2.0.0
+
+Diverged the gem from Shipstation to the Bactracs API
+

data/app/jobs/solidus_backtracs/api/schedule_shipment_syncs_job.rb

@@ -20,7 +20,14 @@ module SolidusBacktracs
       end
 
       def query_shipments
-        shipments = SolidusBacktracs::Shipment::PendingApiSyncQuery.apply(::Spree::Shipment.joins(inventory_units: [:variant]).where("spree_variants.sku" => SolidusBacktracs.config.shippable_skus).where("spree_shipments.state" => :ready).where.not(shipstation_order_id: nil).distinct)
+        shipments = SolidusBacktracs::Shipment::PendingApiSyncQuery.apply(
+          ::Spree::Shipment
+            .joins(inventory_units: [:variant])
+            .where("spree_variants.sku" => SolidusBacktracs.config.shippable_skus)
+            .where("spree_shipments.state" => :ready)
+            .where(backtracs_synced_at: nil)
+            .distinct
+        )
       end
     end
   end

data/lib/generators/solidus_backtracs/install/templates/initializer.rb

@@ -11,6 +11,7 @@ SolidusBacktracs.configure do |config|
 
   ## API Configuration
   config.api_base = ENV['BACKTRACS_API_BASE'] || 'https://bactracstest.andlor.com'
+  config.api_retries = ENV['BACKTRACS_API_RETRIES'] || 3
 
   # Backtracs expects the endpoint to be protected by HTTP Basic Auth.
   # Set the username and password you desire for Backtracs to use.
@@ -32,7 +33,7 @@ SolidusBacktracs.configure do |config|
   config.sku_map = {}
   config.default_rma_type = "W"
   config.default_carrier = "FedExGrnd"
-  config.default_ship_method = "GROUND"  
+  config.default_ship_method = "GROUND"
   config.default_status = "OPEN"
   config.default_rp_location = "FG-NEW"
   config.shippable_skus = []

data/lib/solidus_backtracs/api/request_runner.rb

@@ -10,27 +10,41 @@ module SolidusBacktracs
         @username = SolidusBacktracs.configuration.authentication_username
         @password = SolidusBacktracs.configuration.authentication_password
         @api_base = SolidusBacktracs.configuration.api_base
+        @retries  = SolidusBacktracs.configuration.api_retries
       end
 
-      def authenticated_call(method: nil, path: nil, serializer: nil, shipment: nil)
-        unless @username.present? || @password.present? || @api_base.present?
-          raise "Credentials not defined for Authentication"
-        end 
+      def authenticated_call(method: nil, path: nil, serializer: nil, shipment: nil, count: 0)
+        if count <= @retries
+          unless @username.present? || @password.present? || @api_base.present?
+            raise "Credentials not defined for Authentication"
+          end
 
-        Rails.cache.fetch("backtracks_cache_key", expires_in: 1.hour) do
-          @response = self.call(method: :get, path: "/webservices/user/Authentication.asmx/Login?sUserName=#{@username}&sPassword=#{@password}")
-        end
-
-        authenticted = parse_authentication_response(@response, "Result")
+          @response = Rails.cache.fetch("backtracks_cache_key", expires_in: 1.hour, skip_nil: true) do
+            self.call(method: :get, path: "/webservices/user/Authentication.asmx/Login?sUserName=#{@username}&sPassword=#{@password}")
+          end
 
-        if authenticted == 'false'
-          raise RequestError.from_response(@response)
-        else
+          raise RequestError.from_response(@response) unless @response # just try again for @retries?
+          authenticted = parse_authentication_response(@response, "Result")
+          raise RequestError.from_response(@response) if authenticted == "false"
           sguid = parse_authentication_response(@response, "Message")
-          params = serializer.call(shipment, sguid)
 
-          rma_response = call(method: :post, path: path, params: params)
-          sync_shipment(shipment, rma_response)
+          if authenticted == 'false'
+            clear_cache
+            raise "User Not Authenticated"
+          else
+            sguid = parse_authentication_response(@response, "Message")
+            params = serializer.call(shipment, sguid)
+
+            rma_response = call(method: :post, path: path, params: params)
+            unless parse_rma_creation_response(rma_response) == 'true'
+              clear_cache
+              count += 1
+              self.authenticated_call(method: :post, path: '/webservices/rma/rmaservice.asmx', serializer: serializer, shipment: shipment, count: count)
+            end
+            shipment_synced(shipment)
+          end
+        else
+          shipment_sync_failed(shipment)
         end
       end
 
@@ -62,25 +76,33 @@ module SolidusBacktracs
         end
       end
 
+      def clear_cache
+        Rails.cache.delete('backtracks_cache_key')
+        @response = nil
+      end
+
       def parse_authentication_response(response, type)
         response.dig("AuthenticationResponse", type)
       end
 
-      def sync_shipment(shipment, response)
-        result = response.dig("Envelope", "Body", "CreateNewResponse", "CreateNewResult", "Result")
-        if result == 'true'
-          shipment.update_column(:backtracs_synced_at, Time.zone.now)
+      def parse_rma_creation_response(response)
+        response.dig("Envelope", "Body", "CreateNewResponse", "CreateNewResult", "Result")
+      end
 
-          ::Spree::Event.fire(
-            'solidus_backtracs.api.sync_completed',
-            shipment: shipment
-          )
-        else
-          ::Spree::Event.fire(
-            'solidus_backtracs.api.sync_failed',
-            shipment: shipment
-          )
-        end
+      def shipment_synced(shipment)
+        shipment.update_column(:backtracs_synced_at, Time.zone.now)
+
+        ::Spree::Event.fire(
+          'solidus_backtracs.api.sync_completed',
+          shipment: shipment
+        )
+      end
+
+      def shipment_sync_failed(shipment)
+        ::Spree::Event.fire(
+          'solidus_backtracs.api.sync_failed',
+          shipment: shipment
+        )
       end
     end
   end

data/lib/solidus_backtracs/configuration.rb

@@ -19,7 +19,8 @@ module SolidusBacktracs
       :shipment_notice_class,
       :authentication_username,
       :authentication_password,
-      :api_base, 
+      :api_base,
+      :api_retries,
       :proxy_address,
       :proxy_port,
       :proxy_username,

data/lib/solidus_backtracs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SolidusBacktracs
-  VERSION = '2.2.0'
+  VERSION = '2.2.3'
 end

data/{solidus_shipstation.gemspec → solidus_bactracs.gemspec}

@@ -5,16 +5,16 @@ require_relative 'lib/solidus_backtracs/version'
 Gem::Specification.new do |spec|
   spec.name = 'solidus_backtracs'
   spec.version = SolidusBacktracs::VERSION
-  spec.authors = ['Stephen Puiszis']
-  spec.email = 'steve@tablexi.com'
+  spec.authors = ['Zeryab Ali']
+  spec.email = 'zkhan@suvie.com'
 
   spec.summary = 'A Solidus extension for integrating the Backtracs API.'
-  spec.homepage = 'https://github.com/solidusio-contrib/solidus_backtracs'
+  spec.homepage = 'https://github.com/suvie-eng/solidus_backtracs'
   spec.license = 'BSD-3-Clause'
 
   spec.metadata['homepage_uri'] = spec.homepage
-  spec.metadata['source_code_uri'] = 'https://github.com/solidusio-contrib/solidus_backtracs'
-  spec.metadata['changelog_uri'] = 'https://github.com/solidusio-contrib/solidus_backtracs/blob/master/CHANGELOG.md'
+  spec.metadata['source_code_uri'] = 'https://github.com/suvie-eng/solidus_backtracs/'
+  spec.metadata['changelog_uri'] = 'https://github.com/suvie-eng/solidus_backtracs/blob/master/CHANGELOG.md'
 
   spec.required_ruby_version = Gem::Requirement.new('>= 2.5')
 

data/spec/queries/solidus_backtracs/shipment/pending_api_sync_query_spec.rb

@@ -19,15 +19,15 @@ RSpec.describe SolidusBacktracs::Shipment::PendingApiSyncQuery do
         stub_configuration(api_sync_threshold: 10.minutes)
         shipment = create(:order_ready_to_ship).shipments.first.tap do |s|
           s.order.update_columns(updated_at: 7.minutes.ago)
-          s.update_columns(shipstation_synced_at: 8.minutes.ago)
+          s.update_columns(backtracs_synced_at: 8.minutes.ago)
         end
         create(:order_ready_to_ship).shipments.first.tap do |s|
           s.order.update_columns(updated_at: 9.minutes.ago)
-          s.update_columns(shipstation_synced_at: 8.minutes.ago)
+          s.update_columns(backtracs_synced_at: 8.minutes.ago)
         end
         create(:order_ready_to_ship).shipments.first.tap do |s|
           s.order.update_columns(updated_at: 11.minutes.ago)
-          s.update_columns(shipstation_synced_at: 10.minutes.ago)
+          s.update_columns(backtracs_synced_at: 10.minutes.ago)
         end
 
         expect(described_class.apply(Spree::Shipment.all)).to match_array([shipment])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_backtracs
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.3
 platform: ruby
 authors:
-- Stephen Puiszis
+- Zeryab Ali
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-05-13 00:00:00.000000000 Z
+date: 2022-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -115,7 +115,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 description: 
-email: steve@tablexi.com
+email: zkhan@suvie.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -174,7 +174,7 @@ files:
 - lib/solidus_backtracs/shipment_notice.rb
 - lib/solidus_backtracs/testing_support/factories.rb
 - lib/solidus_backtracs/version.rb
-- solidus_shipstation.gemspec
+- solidus_bactracs.gemspec
 - spec/controllers/spree/backtracs_controller_spec.rb
 - spec/fixtures/backtracs_xml_schema.xsd
 - spec/jobs/solidus_backtracs/api/schedule_shipment_syncs_job_spec.rb
@@ -197,13 +197,13 @@ files:
 - spec/support/controllers.rb
 - spec/support/webmock.rb
 - spec/support/xsd.rb
-homepage: https://github.com/solidusio-contrib/solidus_backtracs
+homepage: https://github.com/suvie-eng/solidus_backtracs
 licenses:
 - BSD-3-Clause
 metadata:
-  homepage_uri: https://github.com/solidusio-contrib/solidus_backtracs
-  source_code_uri: https://github.com/solidusio-contrib/solidus_backtracs
-  changelog_uri: https://github.com/solidusio-contrib/solidus_backtracs/blob/master/CHANGELOG.md
+  homepage_uri: https://github.com/suvie-eng/solidus_backtracs
+  source_code_uri: https://github.com/suvie-eng/solidus_backtracs/
+  changelog_uri: https://github.com/suvie-eng/solidus_backtracs/blob/master/CHANGELOG.md
 post_install_message: 
 rdoc_options: []
 require_paths: