Skip to content

fix(app): clear viewed session notifications#33574

Merged
Brendonovich merged 1 commit into
anomalyco:devfrom
Brendonovich:clear-session-notifications
Jun 24, 2026
Merged

fix(app): clear viewed session notifications#33574
Brendonovich merged 1 commit into
anomalyco:devfrom
Brendonovich:clear-session-notifications

Conversation

@Brendonovich

Copy link
Copy Markdown
Member

Summary

  • mark existing session notifications viewed when the new layout opens a session route
  • wait for persisted notification state before clearing unread indexes
  • restore the route-entry behavior provided by the legacy layout

Testing

  • bun typecheck from packages/app
  • bun test --preload ./happydom.ts ./src/utils/session-route.test.ts ./src/utils/notification-click.test.ts
  • pre-push repository typecheck (23 packages)

@Brendonovich Brendonovich requested a review from Hona as a code owner June 24, 2026 02:20
@Brendonovich Brendonovich merged commit 51eb87b into anomalyco:dev Jun 24, 2026
10 checks passed
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.

1 participant