فهرست منبع

Merge pull request #4171 from weseek/fix/slack-review-for-unuse-home-tab

Fix/slack review for unuse home tab
Yuki Takei 4 سال پیش
والد
کامیت
9f7a8181dc
2فایلهای تغییر یافته به همراه6 افزوده شده و 2 حذف شده
  1. 3 0
      packages/slack/src/utils/publish-initial-home-view.ts
  2. 3 2
      packages/slackbot-proxy/src/controllers/slack.ts

+ 3 - 0
packages/slack/src/utils/publish-initial-home-view.ts

@@ -1,3 +1,6 @@
+// Now Home tab is disabled
+// TODO Imple Home tab
+
 import { ViewsPublishResponse, WebClient } from '@slack/web-api';
 
 export const publishInitialHomeView = (client: WebClient, userId: string): Promise<ViewsPublishResponse> => {

+ 3 - 2
packages/slackbot-proxy/src/controllers/slack.ts

@@ -10,7 +10,7 @@ import { Installation } from '@slack/oauth';
 
 import {
   markdownSectionBlock, GrowiCommand, parseSlashCommand, postEphemeralErrors, verifySlackRequest, generateWebClient,
-  InvalidGrowiCommandError, requiredScopes, postWelcomeMessage, publishInitialHomeView, REQUEST_TIMEOUT_FOR_PTOG,
+  InvalidGrowiCommandError, requiredScopes, postWelcomeMessage, REQUEST_TIMEOUT_FOR_PTOG,
 } from '@growi/slack';
 
 import { Relation } from '~/entities/relation';
@@ -395,7 +395,8 @@ export class SlackCtrl {
           // post message
           postWelcomeMessage(client, userId),
           // publish home
-          publishInitialHomeView(client, userId),
+          // TODO When Home tab show off, use bellow.
+          // publishInitialHomeView(client, userId),
         ]);
       }
     }