Skip to content

[$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

Closed
jmgasper opened this issue Mar 28, 2021 · 7 comments
Closed

[$200] In app notification updates #506

jmgasper opened this issue Mar 28, 2021 · 7 comments

Comments

@jmgasper
Copy link
Collaborator

Currently

One of our testers found that there are times when they receive multiple notifications it can take up the whole screen.

Screen Shot 2021-03-29 at 10 05 34 am

Expected behaviour

Screen Shot 2021-03-29 at 10 06 01 am

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

@jmgasper
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been created for this ticket.

This is an automated message for ghostar via Topcoder X

@jmgasper
Copy link
Collaborator Author

Challenge https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been assigned to obog.

This is an automated message for ghostar via Topcoder X

@jmgasper jmgasper changed the title [$150] In app notification updates [$200] In app notification updates Mar 28, 2021
@atelomycterus
Copy link
Collaborator

@jmgasper Let me know if you want to replace commented on with comments on as displayed in MarvelApp. The phrases are stored in DB. There is no UI to change them. The list of all phrases in the Phrases section

Please apply PRs:
topcoder-platform/forums-theme#34
topcoder-platform/forums-plugins#80
#516

Thanks!

Notifications

Only allow max 5 notifications at once. If there are more than 5 notifications, fade out and fade in next set.

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).

Phrases

image

Testing

Please clear browser cache before testing.

Desktop:
image

image

Mobile:
image

@jmgasper
Copy link
Collaborator Author

jmgasper commented Apr 1, 2021

@atelomycterus - I think we're fine to leave as commented on for now, thanks!

@jmgasper
Copy link
Collaborator Author

jmgasper commented Apr 1, 2021

Payment task has been updated: https://www.topcoder.com/challenges/37c5c201-c7af-49d8-9ea7-815dcb614fd1
Payments Complete
Winner: obog
Copilot: ghostar
Challenge 37c5c201-c7af-49d8-9ea7-815dcb614fd1 has been paid and closed.

This is an automated message for ghostar via Topcoder X

@sdgun
Copy link
Collaborator

sdgun commented Apr 6, 2021

Verified in Dev.

image

@sdgun
Copy link
Collaborator

sdgun commented Apr 9, 2021

Verified in production.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants