Skip to content

feat(notification): better collapse + testing#904

Merged
seanaye merged 5 commits intomainfrom
seanaye/feat/better-notification-collapse
Jan 12, 2026
Merged

feat(notification): better collapse + testing#904
seanaye merged 5 commits intomainfrom
seanaye/feat/better-notification-collapse

Conversation

@seanaye
Copy link
Copy Markdown
Contributor

@seanaye seanaye commented Jan 9, 2026

Summary

Please review commit by commit

  • clear notifications on apns key
  • write collapse key to db in queue
  • test db calls
  • test notification sending
  • test collapse key

Screenshots, GIFs, and Videos

@seanaye seanaye requested a review from a team as a code owner January 9, 2026 21:57
@claude
Copy link
Copy Markdown
Contributor

claude bot commented Jan 9, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@seanaye seanaye force-pushed the seanaye/feat/better-notification-collapse branch 2 times, most recently from 94c5355 to ef06c8b Compare January 12, 2026 15:06
whutchinson98
whutchinson98 approved these changes Jan 12, 2026
View reviewed changes
@seanaye seanaye force-pushed the seanaye/feat/better-notification-collapse branch from ef06c8b to 504bb02 Compare January 12, 2026 15:49
@seanaye seanaye merged commit e2f4c78 into main Jan 12, 2026
38 checks passed
@seanaye seanaye deleted the seanaye/feat/better-notification-collapse branch January 12, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@whutchinson98 whutchinson98 whutchinson98 approved these changes

Assignees

@seanaye seanaye

Labels

cloud-storage

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@seanaye @whutchinson98