allow to update deal without passing deal value

Author: efedasz

lib/basecrm/services/deals_service.rb

@@ -133,7 +133,7 @@ def extract_params!(deal, *args)
 
     def sanitize(deal)
       deal_hash = deal.to_h.select { |k, _| OPTS_KEYS_TO_PERSIST.include?(k) }
-      deal_hash[:value] = Coercion.to_string(deal_hash[:value])
+      deal_hash[:value] = Coercion.to_string(deal_hash[:value]) if deal_hash[:value]
       deal_hash
     end
   end

spec/services/deals_service_spec.rb

@@ -52,6 +52,11 @@
       @deal = create(:deal)
       expect(client.deals.update(@deal)).to be_instance_of BaseCRM::Deal
     end
+
+    it "allows to skip deal value in payload" do
+      @deal = create(:deal, value: nil)
+      expect(client.deals.update(@deal)).to be_instance_of BaseCRM::Deal
+    end
   end
 
   describe :destroy do