Merge pull request #47 from dabapps/fix-add-collections-item

Fix addCollectionItem
dabapps · Sep 5, 2019 · 9b49099 · 9b49099
2 parents a71d42b + 04c2247
commit 9b49099
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@dabapps/redux-api-collections",
-  "version": "0.4.4",
+  "version": "0.4.5",
   "description": "Type-safe helpers for dealing with Rest-Framework backed collections in Typescript",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

diff --git a/src/collections/reducers.ts b/src/collections/reducers.ts
@@ -100,7 +100,7 @@ export function addCollectionItem<T extends IdKeyedMap<T>>(
         subgroup
       );
       const results = existingCollection.results.concat([
-        recordBuilder(action.payload),
+        recordBuilder(action.payload.data),
       ]);
       const updatedCollection = {
         ...existingCollection,

diff --git a/tests/collections.ts b/tests/collections.ts
@@ -369,9 +369,11 @@ describe('Collections', () => {
       const data2 = collections.reducers.collectionsReducer(
         data,
         addItemSuccess('llamas', '', {
-          furLength: 10,
-          id: '2',
-          name: 'Pajama',
+          data: {
+            furLength: 10,
+            id: '2',
+            name: 'Pajama',
+          },
         })
       );
       const subCollection = getCollectionByName(data2, 'llamas');