refactor(makeactioncreator): set actionCreator's data under 'payload'…

… property improves #35
aneurysmjs · Sep 26, 2019 · 80606c9 · 80606c9
1 parent 0c1e9c3
commit 80606c9
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/app/store/helpers/makeActionCreator/makeActionCreator.js b/src/app/store/helpers/makeActionCreator/makeActionCreator.js
@@ -8,10 +8,15 @@ type ActionCreatorType = <T>(T | Array<T>) => ActionType;
 function makeActionCreator(type: string, ...argNames: Array<string>): ActionCreatorType {
   return function actionCreator(...args) {
     const action = { type };
-    argNames.forEach((arg, index) => {
-      action[argNames[index]] = args[index];
-    });
-    return action;
+    return argNames.reduce((current, arg, index) => {
+      const currentAction = {
+        ...current,
+        payload: {
+          [argNames[index]]: args[index],
+        },
+      };
+      return currentAction;
+    }, action);
   };
 }