-
Notifications
You must be signed in to change notification settings - Fork 0
[$200] In app notification updates #506
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
Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been created for this ticket. |
Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been assigned to obog. |
@jmgasper Let me know if you want to replace Please apply PRs: Thanks! Notifications
No changes. Max 5 notifications are displayed by default. If user is in the middle of a visit only get very recent notifications (DateUpdated > -5 mins ago). PhrasesTestingPlease clear browser cache before testing. |
@atelomycterus - I think we're fine to leave as |
Payment task has been updated: https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 |
Currently
One of our testers found that there are times when they receive multiple notifications it can take up the whole screen.
Expected behaviour
Remove Content Preview.
Ex: for the bottom notification it should only read:
Forum beta Test - INVITE ONLY: holysyid comments on Sample Dataset for this challenge.
Max notifications
Only allow max 5 notifications at once. If there are more than 5 notifications, fade out and fade in next set.
Marvel
The MarvelApp has been updated to address: https://marvelapp.com/prototype/c06bg41/screen/77911797
The text was updated successfully, but these errors were encountered: