Skip to content

Keep created notifications unread#3348

Open
qingfeng312 wants to merge 1 commit into
SecureBananaLabs:mainfrom
qingfeng312:qingfeng/notification-read-default-3347
Open

Keep created notifications unread#3348
qingfeng312 wants to merge 1 commit into
SecureBananaLabs:mainfrom
qingfeng312:qingfeng/notification-read-default-3347

Conversation

@qingfeng312
Copy link
Copy Markdown

/claim #3347

Summary

  • Ensure newly-created notifications always start with read: false.
  • Ignore caller-supplied read values during notification creation.
  • Add a focused API regression test for POST /api/notifications with read: true.

Verification

  • npm ci
  • npm test -w apps/api
  • git diff --check

Payment

Method: USDC
Address: 0xe87b4889baeee4ed60a1b2bfc7b3a6a17bce4ad6
Network: Base

Fixes #3347

github-actions Bot added a commit that referenced this pull request Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Notification creation should ignore client-supplied read state

1 participant