update action name

packages/app/src/components/InAppNotification/InAppNotification.tsx

@@ -55,17 +55,16 @@ export const InAppNotification = (props: Props): JSX.Element => {
     );
   };
 
-  const componentName = `${notification.targetModel}:${notification.action}`;
-  const propsNew = {
-    actionUsers: getActionUsers(),
-    ...props,
-  };
-
   const renderInAppNotificationContent = (): JSX.Element => {
-    switch (componentName) {
-      case 'Page:UPDATE':
+    const propsNew = {
+      actionUsers: getActionUsers(),
+      ...props,
+    };
+    const action: string = notification.action;
+    switch (action) {
+      case 'PAGE_UPDATE':
         return <PageUpdateNotification {...propsNew} onClick={props.onClick(props.notification)} />;
-      case 'Page:COMMENT':
+      case 'COMMENT_CREATE':
         return <PageCommentNotification {...propsNew} onClick={props.onClick(props.notification)} />;
       default:
         return <></>;

packages/app/src/server/models/comment.js

@@ -73,7 +73,7 @@ module.exports = function(crowi) {
       { $set: { comment, isMarkdown } },
     );
 
-    await commentEvent.emit('update');
+    await commentEvent.emit('update', commentData);
 
     return commentData;
   };

packages/app/src/server/service/comment.ts

@@ -35,7 +35,8 @@ class CommentService {
         const Page = getModelSafely('Page') || require('../models/page')(this.crowi);
         await Page.updateCommentCount(savedComment.page);
 
-        await this.createAndSendNotifications(savedComment);
+        const activity = await this.createActivity(savedComment, ActivityDefine.ACTION_COMMENT_CREATE);
+        await this.createAndSendNotifications(activity);
       }
       catch (err) {
         logger.error('Error occurred while handling the comment create event:\n', err);
@@ -44,9 +45,10 @@ class CommentService {
     });
 
     // update
-    this.commentEvent.on('update', async() => {
+    this.commentEvent.on('update', async(updatedComment) => {
       try {
         this.commentEvent.onUpdate();
+        await this.createActivity(updatedComment, ActivityDefine.ACTION_COMMENT_UPDATE);
       }
       catch (err) {
         logger.error('Error occurred while handling the comment update event:\n', err);
@@ -67,18 +69,20 @@ class CommentService {
     });
   }
 
-  private createAndSendNotifications = async function(comment) {
-
-    // Create activity
+  private createActivity = async function(comment, action) {
     const parameters = {
       user: comment.creator,
       targetModel: ActivityDefine.MODEL_PAGE,
       target: comment.page,
       eventModel: ActivityDefine.MODEL_COMMENT,
       event: comment._id,
-      action: ActivityDefine.ACTION_COMMENT,
+      action,
     };
     const activity = await this.activityService.createByParameters(parameters);
+    return activity;
+  };
+
+  private createAndSendNotifications = async function(activity) {
 
     // Get user to be notified
     let targetUsers: Types.ObjectId[] = [];

packages/app/src/server/service/page.js

@@ -769,7 +769,7 @@ class PageService {
       user: user._id,
       targetModel: ActivityDefine.MODEL_PAGE,
       target: page,
-      action: ActivityDefine.ACTION_UPDATE,
+      action: ActivityDefine.ACTION_PAGE_UPDATE,
     };
     const activity = await activityService.createByParameters(parameters);
 

packages/app/src/server/util/activityDefine.ts

@@ -1,11 +1,15 @@
 const MODEL_PAGE = 'Page';
 const MODEL_COMMENT = 'Comment';
 
-const ACTION_UPDATE = 'UPDATE';
-const ACTION_COMMENT = 'COMMENT';
-const ACTION_CREATE = 'CREATE'; // Not support yet
-const ACTION_DELETE = 'DELETE'; // Not support yet
-const ACTION_LIKE = 'LIKE'; // Not support yet
+const ACTION_PAGE_UPDATE = 'PAGE_UPDATE';
+const ACTION_COMMENT_CREATE = 'COMMENT_CREATE';
+const ACTION_COMMENT_UPDATE = 'COMMENT_UPDATE';
+
+// const ACTION_UPDATE = 'UPDATE';
+// const ACTION_COMMENT = 'COMMENT';
+// const ACTION_CREATE = 'CREATE'; // Not support yet
+// const ACTION_DELETE = 'DELETE'; // Not support yet
+// const ACTION_LIKE = 'LIKE'; // Not support yet
 
 const getSupportTargetModelNames = () => {
   return [MODEL_PAGE];
@@ -17,10 +21,12 @@ const getSupportEventModelNames = () => {
 
 const getSupportActionNames = () => {
   return [
-    // ACTION_CREATE,
-    ACTION_UPDATE,
+    ACTION_PAGE_UPDATE,
+    ACTION_COMMENT_CREATE,
+    ACTION_COMMENT_UPDATE,
+    // ACTION_UPDATE,
     // ACTION_DELETE,
-    ACTION_COMMENT,
+    // ACTION_COMMENT,
     // ACTION_LIKE,
   ];
 };
@@ -29,11 +35,15 @@ const activityDefine = {
   MODEL_PAGE,
   MODEL_COMMENT,
 
-  ACTION_CREATE, // Not support yet
-  ACTION_UPDATE,
-  ACTION_DELETE, // Not support yet
-  ACTION_COMMENT,
-  ACTION_LIKE,
+  ACTION_PAGE_UPDATE,
+  ACTION_COMMENT_CREATE,
+  ACTION_COMMENT_UPDATE,
+
+  // ACTION_CREATE, // Not support yet
+  // ACTION_UPDATE,
+  // ACTION_DELETE, // Not support yet
+  // ACTION_COMMENT,
+  // ACTION_LIKE,
 
   getSupportTargetModelNames,
   getSupportEventModelNames,