Merge pull request #862 from weseek/fix/cannot-notify-to-Slack

bug/cannot notify to slack

src/server/routes/page.js

@@ -94,6 +94,7 @@ module.exports = function(crowi, app) {
 
     if (crowi.slack) {
       const promises = slackChannels.split(',').map(function(chan) {
+        logger.debug('crowi.slack is exist', crowi.slack);
         return crowi.slack.postPage(page, user, chan, updateOrCreate, previousRevision);
       });
 

src/server/util/slack.js

@@ -128,7 +128,7 @@ module.exports = function(crowi) {
       author_link: urljoin(url, 'user', user.username),
       author_icon: user.image,
       title: page.path,
-      title_link: urljoin(url, page._id),
+      title_link: urljoin(url, page.id),
       text: body,
       mrkdwn_in: ['text'],
     };