From 069d054e30bcc4f917c1a5cb1667cb613634a144 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Wed, 23 Oct 2024 19:47:12 +0200 Subject: [PATCH] remove console logs and lint --- .../notificationPopup/notificationPopup.jsx | 19 ++++++++----------- server/admin.api.js | 2 -- 2 files changed, 8 insertions(+), 13 deletions(-) diff --git a/client/homebrew/brewRenderer/notificationPopup/notificationPopup.jsx b/client/homebrew/brewRenderer/notificationPopup/notificationPopup.jsx index 9c5b5f086..3ea165d62 100644 --- a/client/homebrew/brewRenderer/notificationPopup/notificationPopup.jsx +++ b/client/homebrew/brewRenderer/notificationPopup/notificationPopup.jsx @@ -7,7 +7,6 @@ import Dialog from '../../../components/dialog.jsx'; const DISMISS_BUTTON = ; const NotificationPopup = ()=>{ - const [notifications, setNotifications] = useState([]); const [dissmissKeyList, setDismissKeyList] = useState([]); const [error, setError] = useState(null); @@ -16,9 +15,8 @@ const NotificationPopup = ()=>{ getNotifications(); }, []); - const getNotifications = async () => { + const getNotifications = async ()=>{ setError(null); - try { const res = await request.get('/admin/notification/all'); pickActiveNotifications(res.body || []); @@ -26,30 +24,29 @@ const NotificationPopup = ()=>{ console.log(err); setError(`Error looking up notifications: ${err?.response?.body?.message || err.message}`); } - } + }; - const pickActiveNotifications = (notifs) => { + const pickActiveNotifications = (notifs)=>{ const now = new Date(); - const filteredNotifications = notifs.filter(notification => { + const filteredNotifications = notifs.filter((notification)=>{ const startDate = new Date(notification.startAt); const stopDate = new Date(notification.stopAt); const dismissed = localStorage.getItem(notification.dismissKey) ? true : false; return now >= startDate && now <= stopDate && !dismissed; }); setNotifications(filteredNotifications); - setDismissKeyList(filteredNotifications.map(notif => notif.dismissKey)); - } + setDismissKeyList(filteredNotifications.map((notif)=>notif.dismissKey)); + }; const renderNotificationsList = ()=>{ - if(error) - return
{error}
; + if(error) return
{error}
; return notifications.map((notification)=>(
  • {notification.title}

  • - )) + )); }; return diff --git a/server/admin.api.js b/server/admin.api.js index 119ec3eb5..fd724b9f1 100644 --- a/server/admin.api.js +++ b/server/admin.api.js @@ -143,7 +143,6 @@ router.get('/admin/stats', mw.adminOnly, async (req, res)=>{ router.get('/admin/notification/all', async (req, res, next)=>{ try { const notifications = await NotificationModel.getAll(); - console.log('notifications: ', notifications); return res.json(notifications); } catch (error) { @@ -153,7 +152,6 @@ router.get('/admin/notification/all', async (req, res, next)=>{ }); router.post('/admin/notification/add', mw.adminOnly, async (req, res, next)=>{ - console.table(req.body); try { const notification = await NotificationModel.addNotification(req.body); return res.status(201).json(notification);