fix: state and timestamp bugs (#528)

* fix: state bugs

* chore: libs

package.json

@@ -16,12 +16,11 @@
     "decentraland-connect": "^5.1.0",
     "decentraland-crypto-fetch": "^2.0.1",
     "decentraland-transactions": "^1.47.0",
-    "decentraland-ui": "^4.30.2",
+    "decentraland-ui": "^4.30.3",
     "ethers": "^5.6.8",
     "events": "^3.3.0",
     "flat": "^5.0.2",
     "pusher-js": "^8.0.1",
-    "radash": "7.1.0",
     "react-intl": "^5.20.7",
     "redux-persistence": "^1.2.0",
     "redux-persistence-engine-localstorage": "^1.0.0",

src/containers/UserInformation/UserInformation.tsx

@@ -1,5 +1,4 @@
 import React, { useCallback, useEffect, useMemo, useState } from 'react'
-import { diff } from 'radash/dist/array'
 import {
   MenuItemType,
   UserInformationContainer as UserMenuComponent
@@ -16,7 +15,7 @@ import {
   DROPDOWN_MENU_SIGN_IN_EVENT,
   DROPDOWN_MENU_SIGN_OUT_EVENT
 } from './constants'
-import { NotificationsAPI, checkIsOnboarding, parseNotification} from '../../modules/notifications'
+import { NotificationsAPI, checkIsOnboarding} from '../../modules/notifications'
 
 const NOTIFICATIONS_QUERY_INTERVAL = 60000;
 
@@ -130,17 +129,10 @@ export const UserInformation = (props: UserInformationProps) => {
   }, [analytics, onSignIn])
 
   const fetchNotificationsState = () => {
-    setUserNotifications({ notifications, isLoading: true })
+    setUserNotifications({ notifications: [], isLoading: true })
     client.getNotifications()
-    .then((response) => {
-      const parsed = response.notifications.map(parseNotification)
-
-      // check if needed to update notifications state
-      if(diff(notifications, parsed).length) {
-        setUserNotifications({ isLoading: false, notifications: parsed })
-      } else {
-        setUserNotifications({ notifications, isLoading: false })
-      }
+    .then((retrievedNotifications) => {
+      setUserNotifications({ isLoading: false, notifications: retrievedNotifications })
     })
   }
 
@@ -160,12 +152,7 @@ export const UserInformation = (props: UserInformationProps) => {
   useEffect(() => {
     if (identity && withNotifications) {
       fetchNotificationsState()
-    }
-  }, [identity])
-
 
-  useEffect(() => {
-    if (identity && withNotifications) {
       const interval = setInterval(() => {
         fetchNotificationsState()
       }, NOTIFICATIONS_QUERY_INTERVAL)
@@ -176,7 +163,7 @@ export const UserInformation = (props: UserInformationProps) => {
     } else {
       return () => {}
     }
-  }, [identity, notifications])
+  }, [identity])
 
   return (
     <UserMenuComponent

src/modules/notifications.ts

@@ -22,9 +22,10 @@ export class NotificationsAPI extends BaseClient {
       params.append("from", `${from}`)
     }
 
-    const notificationsResponse = await this.fetch<{notifications: Array<DCLNotification>}>(`/notifications${params.toString().length ? `?${params.toString()}` : ''}`)
+    const { notifications } = await this
+    .fetch<{notifications: Array<DCLNotification>}>(`/notifications${params.toString().length ? `?${params.toString()}` : ''}`)
 
-    return notificationsResponse
+    return notifications.map(parseNotification)
   }
 
   async markNotificationsAsRead(ids: string[]) {
@@ -48,9 +49,9 @@ export const checkIsOnboarding = () => {
   }
 }
 
-export const parseNotification = (notification: DCLNotification): DCLNotification => {
+const parseNotification = (notification: DCLNotification): DCLNotification => {
   return ({
     ...notification,
-    timestamp: Number(notification.timestamp) * 1000
+    timestamp: Number(notification.timestamp)
   })
 }