kaori 4 лет назад
Родитель
Сommit
1ddf19aafd
1 измененных файлов с 11 добавлено и 6 удалено
  1. 11 6
      packages/app/src/components/InAppNotification/InAppNotificationPage.tsx

+ 11 - 6
packages/app/src/components/InAppNotification/InAppNotificationPage.tsx

@@ -1,4 +1,6 @@
-import React, { FC, useState, useEffect } from 'react';
+import React, {
+  FC, useState, useEffect, useCallback,
+} from 'react';
 
 
 import { useTranslation } from 'react-i18next';
 import { useTranslation } from 'react-i18next';
 import PropTypes from 'prop-types';
 import PropTypes from 'prop-types';
@@ -11,6 +13,10 @@ import CustomNavAndContents from '../CustomNavigation/CustomNavAndContents';
 import { InAppNotificationStatuses } from '~/interfaces/in-app-notification';
 import { InAppNotificationStatuses } from '~/interfaces/in-app-notification';
 import { apiv3Put, apiv3Post } from '~/client/util/apiv3-client';
 import { apiv3Put, apiv3Post } from '~/client/util/apiv3-client';
 
 
+import loggerFactory from '~/utils/logger';
+
+const logger = loggerFactory('growi:InAppNotificationPage');
+
 
 
 type Props = {
 type Props = {
   appContainer: AppContainer
   appContainer: AppContainer
@@ -22,20 +28,19 @@ const InAppNotificationPageBody: FC<Props> = (props) => {
   const { t } = useTranslation();
   const { t } = useTranslation();
   const { mutate: mutateInAppNotificationStatusData } = useSWRxInAppNotificationStatus();
   const { mutate: mutateInAppNotificationStatusData } = useSWRxInAppNotificationStatus();
 
 
-
-  const updateNotificationStatus = async() => {
+  const updateNotificationStatus = useCallback(async() => {
     try {
     try {
       await apiv3Post('/in-app-notification/read');
       await apiv3Post('/in-app-notification/read');
       mutateInAppNotificationStatusData();
       mutateInAppNotificationStatusData();
     }
     }
     catch (err) {
     catch (err) {
-      // logger.error(err);
+      logger.error(err);
     }
     }
-  };
+  }, [mutateInAppNotificationStatusData]);
 
 
   useEffect(() => {
   useEffect(() => {
     updateNotificationStatus();
     updateNotificationStatus();
-  }, []);
+  }, [updateNotificationStatus]);
 
 
   const InAppNotificationCategoryByStatus = (status?: InAppNotificationStatuses) => {
   const InAppNotificationCategoryByStatus = (status?: InAppNotificationStatuses) => {
     const [activePage, setActivePage] = useState(1);
     const [activePage, setActivePage] = useState(1);