Merge pull request #4504 from weseek/feat/79512-open-the-notification-dropdown-to-mark-as-read

Feat/79512 open the notification dropdown to mark as read

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

@@ -45,9 +45,9 @@ const InAppNotificationDropdown: FC<Props> = (props: Props) => {
     });
   };
 
-  const updateNotificationStatus = () => {
+  const updateNotificationStatus = async() => {
     try {
-      // await this.props.crowi.apiPost('/notification.read');
+      await appContainer.apiv3Post('/in-app-notification/read');
       setCount(0);
     }
     catch (err) {

packages/app/src/server/routes/apiv3/in-app-notification.ts

@@ -43,13 +43,12 @@ module.exports = (crowi) => {
     }
   });
 
-  router.post('/read', accessTokenParser, loginRequiredStrictly, csrf, (req, res) => {
+  router.post('/read', accessTokenParser, loginRequiredStrictly, csrf, async(req, res) => {
     const user = req.user;
 
     try {
-      const notification = InAppNotification.read(user);
-      const result = { notification };
-      return res.apiv3(result);
+      await inAppNotificationService.read(user);
+      return res.apiv3();
     }
     catch (err) {
       return res.apiv3Err(err);