Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: add Mark All as Read feature for inbox notifications #16975

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

feat: add Mark All as Read feature for inbox notifications #16975

wants to merge 1 commit into from

Conversation

kylecarbs
Copy link
Member

Summary

Details

This PR adds:

  1. A new SQL query to mark all unread notifications as read for a user
  2. A new REST API endpoint at /api/v2/notifications/inbox/mark-all-read
  3. A new SDK method MarkAllInboxNotificationsRead
  4. A comprehensive test to verify the functionality

Fixes #15264

Test plan

  • Manual testing: Open notifications panel and use mark all as read button
  • Automated tests: Added test in TestInboxNotifications_MarkAllRead which verifies that:
    • All notifications are properly marked as read
    • Unread count is properly updated to 0

🤖 Generated with Claude Code

feat: add Mark All as Read feature for inbox notifications
b88f430 
Implements feature request from issue #15264 to add a "Mark all as read" feature for notifications. This PR adds:
1. A new SQL query to mark all unread notifications as read for a user
2. A new REST API endpoint at /api/v2/notifications/inbox/mark-all-read
3. A new SDK method MarkAllInboxNotificationsRead
4. A comprehensive test to verify the functionality

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@github-actions GitHub Actions
Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

Claude Anthropic seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions bot added the stale This issue is like stale bread. label Mar 26, 2025
@github-actions github-actions bot closed this Mar 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@kylecarbs kylecarbs

Labels
stale This issue is like stale bread.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Provide Postgresql database settings not only via connection string
1 participant
@kylecarbs