@friggframework/core 1.1.4--canary.301.abb73c7.0 → 1.1.4--canary.301.ffb9dc2.0

package/integrations/integration-router.js

@@ -89,12 +89,9 @@ function setIntegrationRoutes(router, factory, getUserId) {
             );
             await integration.onCreate();
 
-            // filtered set for results
-            const response = await IntegrationHelper.getFormattedIntegration(
-                integration.record
+            res.status(201).json(
+                await IntegrationHelper.getFormattedIntegration(integration.record)
             );
-            res.status(201);
-            res.json(response);
         })
     );
 
@@ -114,11 +111,9 @@ function setIntegrationRoutes(router, factory, getUserId) {
             );
             await integration.onUpdate(params);
 
-            const response = await IntegrationHelper.getFormattedIntegration(
-                integration.record
+            res.json(
+                await IntegrationHelper.getFormattedIntegration(integration.record)
             );
-
-            res.json(response);
         })
     );
 
@@ -142,8 +137,7 @@ function setIntegrationRoutes(router, factory, getUserId) {
                 params.integrationId
             );
 
-            res.status(201);
-            res.json({});
+            res.status(201).json({});
         })
     );
 
@@ -154,8 +148,7 @@ function setIntegrationRoutes(router, factory, getUserId) {
             ]);
             const integration =
                 await integrationFactory.getInstanceFromIntegrationId(params);
-            const results = await integration.getConfigOptions();
-            res.json(results);
+            res.json(await integration.getConfigOptions());
         })
     );
 
@@ -166,10 +159,10 @@ function setIntegrationRoutes(router, factory, getUserId) {
             ]);
             const integration =
                 await integrationFactory.getInstanceFromIntegrationId(params);
-            const results = await integration.refreshConfigOptions(
-                req.body
+
+            res.json(
+                await integration.refreshConfigOptions(req.body)
             );
-            res.json(results);
         })
     );
 
@@ -181,11 +174,10 @@ function setIntegrationRoutes(router, factory, getUserId) {
             ]);
             const integration =
                 await integrationFactory.getInstanceFromIntegrationId(params);
-            const results = await integration.getActionOptions(
-                params.actionId
+
+            res.json(
+                await integration.getActionOptions(params.actionId)
             );
-            // We could perhaps augment router with dynamic options? Haven't decided yet, but here may be the place
-            res.json(results);
         })
     );
 
@@ -197,12 +189,10 @@ function setIntegrationRoutes(router, factory, getUserId) {
             ]);
             const integration =
                 await integrationFactory.getInstanceFromIntegrationId(params);
-            const results = await integration.refreshActionOptions(
-                params.actionId,
-                req.body
+
+            res.json(
+                await integration.refreshActionOptions(params.actionId, req.body)
             );
-            // We could perhaps augment router with dynamic options? Haven't decided yet, but here may be the place
-            res.json(results);
         })
     );
 
@@ -214,14 +204,12 @@ function setIntegrationRoutes(router, factory, getUserId) {
             ]);
             const integration =
                 await integrationFactory.getInstanceFromIntegrationId(params);
-            const results = await integration.notify(
-                params.actionId,
-                req.body
+
+            res.json(
+                await integration.notify(params.actionId, req.body)
             );
-            // We could perhaps augment router with dynamic options? Haven't decided yet, but here may be the place
-            res.json(results);
         })
-    )
+    );
 
     router.route('/api/integrations/:integrationId').get(
         catchAsyncError(async (req, res) => {
@@ -298,6 +286,7 @@ function setEntityRoutes(router, factory, getUserId) {
                     `Error: EntityManager of type ${params.entityType} requires a valid url`
                 );
             }
+
             res.json(await module.getAuthorizationRequirements());
         })
     );
@@ -308,15 +297,14 @@ function setEntityRoutes(router, factory, getUserId) {
                 'entityType',
                 'data',
             ]);
-            console.log('post authorize', params);
             const module = await getModuleInstance(req, params.entityType);
-            console.log('post authorize module', module);
-            const results = await module.processAuthorizationCallback({
-                userId: getUserId(req),
-                data: params.data,
-            });
 
-            res.json(results);
+            res.json(
+                await module.processAuthorizationCallback({
+                    userId: getUserId(req),
+                    data: params.data,
+                })
+            );
         })
     );
 
@@ -382,6 +370,7 @@ function setEntityRoutes(router, factory, getUserId) {
             if (!module) {
                 throw Boom.notFound();
             }
+
             res.json(module.entity);
         })
     );

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/core",
   "prettier": "@friggframework/prettier-config",
-  "version": "1.1.4--canary.301.abb73c7.0",
+  "version": "1.1.4--canary.301.ffb9dc2.0",
   "dependencies": {
     "@hapi/boom": "^10.0.1",
     "aws-sdk": "^2.1200.0",
@@ -15,9 +15,9 @@
     "node-fetch": "^2.6.7"
   },
   "devDependencies": {
-    "@friggframework/eslint-config": "1.1.4--canary.301.abb73c7.0",
-    "@friggframework/prettier-config": "1.1.4--canary.301.abb73c7.0",
-    "@friggframework/test": "1.1.4--canary.301.abb73c7.0",
+    "@friggframework/eslint-config": "1.1.4--canary.301.ffb9dc2.0",
+    "@friggframework/prettier-config": "1.1.4--canary.301.ffb9dc2.0",
+    "@friggframework/test": "1.1.4--canary.301.ffb9dc2.0",
     "@types/lodash": "^4.14.191",
     "@typescript-eslint/eslint-plugin": "^5.55.0",
     "chai": "^4.3.6",
@@ -49,5 +49,5 @@
   },
   "homepage": "https://github.com/friggframework/frigg#readme",
   "description": "",
-  "gitHead": "abb73c78a1a3f30241676e3999e61e7e3bda5578"
+  "gitHead": "ffb9dc213cfb9e02635a17234c126a412f734039"
 }