Skip to content

Notifications mark all read - #14416

Open
raghavg1112 wants to merge 2 commits into
TEAMMATES:masterfrom
raghavg1112:master
Open

raghavg1112 wants to merge 2 commits into
TEAMMATES:masterfrom
raghavg1112:master

Conversation

@raghavg1112

Copy link
Copy Markdown

[#13471] Notifications page: mark all as read

Fixes #13471

Outline of Solution

Added a "Mark All as Read" button to the student/instructor notifications page. Clicking it calls a new [/webapi/notifications/read/all] backend endpoint that marks all active notifications as read in one request, instead of one by one. The button is disabled when there are no unread notifications or while the request is in progress.

@github-actions

Copy link
Copy Markdown

Hi @raghavg1112, thank you for your interest in contributing to TEAMMATES!
However, your PR does not appear to follow our contributing guidelines:

  • Title must start with the issue number the PR is fixing in square brackets, e.g. [#<issue-number>]

Please address the above before we proceed to review your PR.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Notifications page: mark all as read

1 participant