Merge pull request #28 from thgrw/tags-translation

Tags translation

Author: nzlosh

CHANGES.md

@@ -1,5 +1,13 @@
 # Change Log
 
+## 3.0.5
+
+- Bug fix: Tags fields need additional translation when modifying 
+
+## 3.0.4
+
+- Bug fix: PATCH requests don't need all the fields to be required 
+
 ## 3.0.3
 
 - Updated spec as per v3.1.7

actions/lib/action.py

@@ -38,7 +38,10 @@ def make_request(self, endpoint_uri, http_action, **kwargs):
 
         # transform `tags` if present
         if kwargs.get("tags"):
-            kwargs["tags"] = ",".join(kwargs["tags"])
+            if http_action in ['post', 'put', 'patch']:
+                kwargs["tags"] = [{"slug": x} for x in kwargs["tags"]]
+            else:
+                kwargs["tags"] = ",".join(kwargs["tags"])
             self.logger.debug("tags transformed to {}".format(kwargs["tags"]))
 
         # strip values which have a None value if we are making a write request

bin/generate.py

@@ -108,7 +108,7 @@ def run(spec):
                 ref_name = verb_data['parameters'][0]['schema']['$ref'].split('/')[-1]
                 schema = spec['definitions'][ref_name]
                 try:
-                    required = schema['required']
+                    required = ['id'] if verb == 'patch' else schema['required']
                 except KeyError:
                     required = []
                 action['parameters'] = parse_properties(schema['properties'], required, spec)

pack.yaml

@@ -6,7 +6,7 @@ keywords:
   - networking
   - ipam
   - dcim
-version: 3.0.3
+version: 3.0.5
 author: John Anderson, Jefferson White
 email: [email protected]
 python_versions: