-
Notifications
You must be signed in to change notification settings - Fork 212
Notification pop-up inconsistent #4140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@Dara-K, from a UI perspective, when should members see the above? |
When they have read all notifications. That's also why the "Mark all as read" link is not there anymore. |
@Oanh-and-only-Oanh , the. issue is members should see the empty state only when they do not have any notifications. But sometimes for the same user, they see both the above randomly. Sometimes it shows the empty state and sometimes with notifications. |
@Oanh-and-only-Oanh @Dara-K @SathyaJayabal Yeah, the issue here is about sometimes the notifications items not loaded. This happen after login if refresh back to normal. I'll check this one. |
@sushilshinde I can't reproduce this one anymore, maybe was API instability? Thanks |
UPDATE Working on fix. |
@SathyaJayabal @sushilshinde |
@luizrrodrigues , verified on beta. The notifications load correctly every time. |
Verified on beta (develop branch) |
verified on prod. |
@luizrrodrigues sometimes I see this pop-up
and sometimes I see this
FYI @Oanh-and-only-Oanh @SathyaJayabal
The text was updated successfully, but these errors were encountered: