Merge pull request #53 from basecrm/v1.3.1

v1.3.1

Author: Erwin Fedasz

CHANGELOG.md

@@ -1,5 +1,10 @@
 ## CHANGELOG
 
+v1.3.1 (2017-12-08)
+**Features and Improvements**
+
+* Fixed issue where could not update deal without passing value
+
 v1.3.0 (2017-10-19)
 **Features and Improvements**
 

VERSION

@@ -1 +1 @@
-1.3.0
+1.3.1

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

lib/basecrm/version.rb

@@ -1,3 +1,3 @@
 module BaseCRM
-  VERSION = "1.3.0"
+  VERSION = "1.3.1"
 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