kaori 5 лет назад
Родитель
Сommit
39fdc011bf

+ 9 - 14
packages/slackbot-proxy/src/controllers/slack.ts

@@ -167,22 +167,17 @@ export class SlackCtrl {
     const payload = JSON.parse(body.payload);
     const { type } = payload;
 
-    try {
-      if (type === 'view_submission') {
-        switch (payload.response_urls[0].action_id) {
-          case 'show_proxy_url':
-            await this.registerService.upsertOrderRecord(this.orderRepository, installation, payload);
-            await this.registerService.showProxyURL(authorizeResult, payload);
-            break;
-          default:
-            break;
-        }
-      }
-    }
-    catch (error) {
-      logger.error(error);
+    // register
+    if (type === 'view_submission' && payload.response_urls[0].action_id === 'show_proxy_url') {
+      await this.registerService.upsertOrderRecord(authorizeResult, this.orderRepository, installation, payload);
+      return;
     }
 
+    /*
+     * forward to GROWI server
+     */
+    // TODO: forward to GROWI server by GW-5866
+
   }
 
   @Post('/events')

+ 4 - 6
packages/slackbot-proxy/src/services/RegisterService.ts

@@ -56,8 +56,10 @@ export class RegisterService implements GrowiCommandProcessor {
     });
   }
 
-  // eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
-  async upsertOrderRecord(orderRepository: OrderRepository, installation: Installation | undefined, payload: any): Promise<void> {
+  async upsertOrderRecord(
+      // eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
+      authorizeResult:AuthorizeResult, orderRepository: OrderRepository, installation: Installation | undefined, payload: any,
+  ): Promise<void> {
     const inputValues = payload.view.state.values;
     const inputGrowiUrl = inputValues.growiDomain.contents_input.value;
     const inputGrowiAccessToken = inputValues.growiAccessToken.contents_input.value;
@@ -75,10 +77,6 @@ export class RegisterService implements GrowiCommandProcessor {
         installation: installation?.id, growiUrl: inputGrowiUrl, growiAccessToken: inputGrowiAccessToken, proxyAccessToken: inputProxyAccessToken,
       });
     }
-  }
-
-  // eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types
-  async showProxyURL(authorizeResult: AuthorizeResult, payload: any): Promise<void> {
 
     // TODO: implement for when proxy URL is undefined by GW-5834
     let proxyURL;