Skip to content

fix: suppress notification sounds and zap animations for historical events#65

Merged
barrydeen merged 1 commit intomainfrom
fix/suppress-historical-notif-sounds
Feb 27, 2026
Merged

fix: suppress notification sounds and zap animations for historical events#65
barrydeen merged 1 commit intomainfrom
fix/suppress-historical-notif-sounds

Conversation

@barrydeen
Copy link
Copy Markdown
Owner

Summary

  • Add notifInitialized flag to NotificationRepository that gates sound and zap animation emissions
  • Reset flag to false at the start of subscribeDmsAndNotifications() and set true after EOSE arrives
  • Prevents flood of notification sounds and stuck zap animations on cold start and force reconnect (30s+ background)

Test plan

  • Force-close app with pending notifications, reopen — zero sounds on startup
  • Wait for feed to load, receive a live reaction/zap — sound plays and animation shows
  • Background app 30+ seconds, reopen — zero sounds during reload, live events trigger normally
  • Switch accounts — flag resets cleanly via clear()

…vents

Gate sound/animation emissions behind a notifInitialized flag that stays
false until EOSE arrives on the notification subscription. Prevents the
flood of sounds on cold start and force reconnect.
@barrydeen barrydeen merged commit 0c09bc8 into main Feb 27, 2026
@barrydeen barrydeen deleted the fix/suppress-historical-notif-sounds branch March 4, 2026 01:29
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