fulfil_api 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a26b32a0feab8bd7e697b27db519ac3945fefcd00159aa714742c40ac5abffe
-  data.tar.gz: e8e5c494d57360d1cacbbea096c6c4cac3d70d504d04f4478a5f5508db658a63
+  metadata.gz: 10fcad8a31843f9c7c15e2c0d7e9ed00331fbc665091f088f6fd7794fae7159a
+  data.tar.gz: 367ed1508c804dc8fffc4da5089683702ca18d35433c5d6ae8744a06fae7bde7
 SHA512:
-  metadata.gz: 1d4b7f5ca1a4bc4e8449cc3b235fd9f3b467e7977d76feaa52fa11dac18a6a34ba1ab6b4a8280183e798e45f4ddfda00e0e2d27534ca467110a3cdec203adc49
-  data.tar.gz: 63ce6d7793de383098af74f7ca3c81ef8730351d7483390f8256f2dcd5cecc5cb3385ff46e9046fe7ed51f62e66f183b3b4abbf7b11f92a8322abe1ef44eeec9
+  metadata.gz: 84f5db4b17bb3c6e0011f30dd5b9a8d9472199a49f2562ad1e2646540e0d08f1d89372e42fc3f4492731c6df52e4343cda4547b20dbdcafc48c388aa176780f1
+  data.tar.gz: fa251dcc692380fa34543b6ae0d82338cc09b9cadd5d2d8875d02a02ee6c49168017808607a7508996f61ab10dc1d389b5915c7269b24792997ba65113071c44

data/lib/fulfil_api/customer_shipment.rb

@@ -10,20 +10,17 @@ module FulfilApi
       # Sets the fulfillment status of the customer shipment on hold
       #
       # @param id_or_ids [String, Integer, Array[String], Array[Integer]] The ID(s) of the customer shipment(s) to hold.
-      # @param note [String] A note to define the reason for holding. (Optional)
-      # @param hold_reason [String] An hold reason ID. (Optional)
+      # @param note [String] A note to define the reason for holding.
       # @return [Boolean] Returns true if hold successfully.
       # @raise [FulfilApi::Error] If an error occurs during holding the customer shipment.
       #
       # @example Hold a customer shipment
-      #   FulfilApi::CustomerShipment.hold(123, note: "Double booking", hold_reason: hold_reason_id)
+      #   FulfilApi::CustomerShipment.hold(123, note: "Double booking")
       #
       # @example Hold multipe customer shipments
-      #   FulfilApi::CustomerShipment.hold([123, 456], note: "Double booking", hold_reason: hold_reason_id)
-      def hold!(id_or_ids, note: nil, hold_reason: nil)
-        FulfilApi.client.put("/model/#{MODEL_NAME}/hold",
-                             body: [[*id_or_ids].flatten, { note: note, hold_reason: hold_reason }.compact_blank])
-
+      #   FulfilApi::CustomerShipment.hold([123, 456], note: "Double booking")
+      def hold!(id_or_ids, note:)
+        FulfilApi.client.put("/model/#{MODEL_NAME}/hold", body: [[*id_or_ids].flatten, note])
         true
       end
 
@@ -40,25 +37,23 @@ module FulfilApi
       #
       # @example Unhold a customer shipment
       #   FulfilApi::CustomerShipment.unhold([123, 456], note: "All clear")
-      def unhold!(id_or_ids, note: nil)
-        FulfilApi.client.put("/model/#{MODEL_NAME}/unhold", body: [[*id_or_ids].flatten, { note: note }.compact_blank])
-
+      def unhold!(id_or_ids, note:)
+        FulfilApi.client.put("/model/#{MODEL_NAME}/unhold", body: [[*id_or_ids].flatten, note])
         true
       end
     end
 
     # Sets the current customer shipment on hold, rescuing any errors that occur and handling them based on error type.
     #
-    # @param note [String] A note to define the reason for holding. (Optional)
-    # @param hold_reason [String] An hold reason ID. (Optional)
+    # @param note [String] A note to define the reason for holding.
     # @return [Boolean] Returns true if hold successfully, otherwise false.
     #
     # @example Holds a customer_shipment
-    #   customer_shipment.hold(note: "Double booking", hold_reason: hold_reason_id)
-    def hold(note: nil, hold_reason: nil)
-      self.class.hold!(id, note: note, hold_reason: hold_reason)
+    #   customer_shipment.hold("Holding the shipment for 30 minutes to allow edits to the order")
+    def hold(note)
+      self.class.hold!(id, note: note)
     rescue FulfilApi::Error => e
-      handle_error(e)
+      handle_exception(e)
       false
     end
 
@@ -68,18 +63,12 @@ module FulfilApi
     # @return [Boolean] Returns true if unhold successfully, otherwise false.
     #
     # @example Unholds a customer_shipment
-    #   customer_shipment.unhold(note: "Double booking")
-    def unhold(note: nil)
+    #   customer_shipment.unhold("Ship out these items to the customer")
+    def unhold(note)
       self.class.unhold!(id, note: note)
     rescue FulfilApi::Error => e
-      handle_error(e)
+      handle_exception(e)
       false
     end
-
-    private
-
-    def handle_error(err)
-      errors.add(code: err.details[:response_status], type: :system, message: err.details[:response_body])
-    end
   end
 end

data/lib/fulfil_api/resource/persistable.rb

@@ -52,14 +52,7 @@ module FulfilApi
       def save
         save!
       rescue FulfilApi::Error => e
-        case (error = JSON.parse(e.details[:response_body]).deep_symbolize_keys!)
-        in { type: "UserError" }
-          errors.add(code: error[:code], type: :user, message: error[:message])
-        in { code: Integer, name: String, description: String }
-          errors.add(code: error[:code], type: :authorization, message: error[:description])
-        end
-
-        self
+        handle_exception(e)
       end
 
       # Saves the current resource, raising an error if it cannot be saved.

data/lib/fulfil_api/resource/relation/query_methods.rb

@@ -67,7 +67,7 @@ module FulfilApi
         # @return [FulfilApi::Resource::Relation] A new {Relation} instance with the conditions applied.
         def where(conditions)
           clone.tap do |relation|
-            relation.conditions << conditions.flatten
+            relation.conditions << conditions
             relation.conditions.uniq!
           end
         end

data/lib/fulfil_api/resource.rb

@@ -72,5 +72,18 @@ module FulfilApi
     private
 
     attr_reader :model_name
+
+    def handle_exception(exception) # rubocop:disable Metrics/AbcSize
+      case (error = JSON.parse(exception.details[:response_body]).deep_symbolize_keys!)
+      in { type: "UserError" }
+        errors.add(code: error[:code], type: :user, message: error[:message])
+      in { code: Integer, name: String, description: String }
+        errors.add(code: error[:code], type: :authorization, message: error[:description])
+      else
+        errors.add(code: exception.details[:response_status], type: :system, message: exception.details[:response_body])
+      end
+
+      self
+    end
   end
 end

data/lib/fulfil_api/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FulfilApi
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fulfil_api
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Stefan Vermaas
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-09-17 00:00:00.000000000 Z
+date: 2024-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport