Merge pull request #112 from JaviCerveraIngram/CPS-82-update_parameters

CPS-82 - Fix to update_parameters

Author: marcserrat

connect/resources/fulfillment_automation.py

@@ -179,12 +179,10 @@ def update_parameters(self, pk, params):
         :return: The server response.
         :rtype: str
         """
-        list_dict = []
-        for _ in params:
-            list_dict.append(_.__dict__ if isinstance(_, Param) else _)
+        mapped_params = [p.json for p in params if isinstance(p, Param)]
         return self._api.put(
             path=pk,
-            json={'asset': {'params': list_dict}},
+            json={'asset': {'params': mapped_params}},
         )[0]
 
     def _update_conversation_if_exists(self, conversation, request_id, obj):

connect/resources/tier_config_automation.py

@@ -119,11 +119,8 @@ def update_parameters(self, pk, params):
         :return: The server response.
         :rtype: str
         """
-        list_dict = []
-        for _ in params:
-            list_dict.append(_.__dict__ if isinstance(_, Param) else _)
-
+        mapped_params = [p.json for p in params if isinstance(p, Param)]
         return self._api.put(
             path=pk,
-            json={'params': list_dict},
+            json={'params': mapped_params},
         )[0]