Skip to content
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

Send back the notifications that have been read #35

Open
raaaahman opened this issue Jan 11, 2021 · 1 comment
Open

Send back the notifications that have been read #35

raaaahman opened this issue Jan 11, 2021 · 1 comment
Labels
[Scope] UI - Notification Hub [Status] Blocked When another issue need to be taken care of first. [Type] Feature Describe a feature to be added to the project

Comments

@raaaahman
Copy link
Collaborator

raaaahman commented Jan 11, 2021

Part of #30

After opening the Notifications Hub, every notifications in the current list should be considered as 'read'. See #35 (comment)

So we need to send a call to the back-end to update their status for the current recipient.

This may need for an endpoint / AJAX script to be defined first.

Blocked by #31 , #37

@raaaahman raaaahman added [Status] Blocked When another issue need to be taken care of first. [Scope] UI - Notification Hub labels Jan 11, 2021
@folletto
Copy link
Collaborator

Note here that's not the case. Per latest design (see #23) items are marked as read either:

  1. By clicking on them
  2. By clicking "Mark all as read"

When the menu opens it should not clear notifications.

@raaaahman raaaahman added the [Type] Feature Describe a feature to be added to the project label Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Scope] UI - Notification Hub [Status] Blocked When another issue need to be taken care of first. [Type] Feature Describe a feature to be added to the project
Projects
None yet
Development

No branches or pull requests

2 participants