Merge pull request #260 from charlottemoremen/dataset_ID_

adjustment to add_input_dataset
Datatamer · Aug 16, 2019 · 319dd28 · 319dd28
2 parents b8410c3 + bc8bab8
commit 319dd28
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@
 
   **BUG FIXES**
   - [#235](https://github.com/Datatamer/unify-client-python/issues/235) Making `AttributeCollection` retrieve attributes directly instead of by streaming
+  - [#234](https://github.com/Datatamer/tamr-client/issues/234) Project's `resource`'s `add_input_dataset` now uses params instead of constructing resource ID manually
   - [#256](https://github.com/Datatamer/unify-client-python/issues/256) Record and published clusters refresh did not use the correct endpoint
 
 ## 0.8.0

diff --git a/tamr_unify_client/project/resource.py b/tamr_unify_client/project/resource.py
@@ -104,9 +104,9 @@ def add_input_dataset(self, dataset):
         :return: HTTP response from the server
         :rtype: :class:`requests.Response`
         """
-        dataset_id = dataset.relative_id.split("/")[-1]
+        params = {"id": dataset.relative_id}
         response = self.client.post(
-            self.api_path + "/inputDatasets" + f"?id={dataset_id}"
+            self.api_path + "/inputDatasets", params=params
         ).successful()
         return response