memcode
memcode copied to clipboard
We shouldn't refresh notifications on every page + fix localStorage
-
Notifications should be renewed without setting
{ status: 'request' }
on subsequent api calls. -
There shouldn't be a residual '1 notification is unread' when we:
- have one notification
- mark it as read
- switch to another page, and see one notification as unread for a few seconds.
This happens because we don't update value in our localStorage whenever amount of unread notifications changes.
Let's put <Notifications/>
state into redux
instead of keeping it in the component.