Skip to content

add notifications to web admin#7

Merged
djanogly merged 1 commit intomainfrom
pr/web-admin-notifs
Mar 4, 2026
Merged

add notifications to web admin#7
djanogly merged 1 commit intomainfrom
pr/web-admin-notifs

Conversation

@djanogly
Copy link
Contributor

@djanogly djanogly commented Mar 4, 2026

This pull request improves the notification cue system for inbox events by introducing a new, more distinctive "bing" sound, synchronizing notification preferences across tabs and components, and displaying a badge for unread inbox messages in the sidebar. It also sets sound notifications to be enabled by default and adds robust event broadcasting and listening for preference changes.

Notification sound improvements:

  • Replaces the old single-tone sound with a new playInboxBingSound function that plays a two-tone "bing" sound, implemented in a new utility (playInboxBingSound.ts), and updates all relevant components to use this function instead of the previous inline implementation. [1] [2] [3] [4]

Notification preferences synchronization:

  • Introduces the INBOX_CUE_PREFERENCES_UPDATED_EVENT event and a broadcastInboxCuePreferencesUpdated function to broadcast inbox cue preference changes across tabs and components, ensuring all listeners are updated immediately. Components now listen for this event in addition to the storage event. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Sidebar unread badge and notification cues:

  • Adds an unread badge for the inbox in the sidebar for admin users, showing the count (capped at "99+"), and plays the notification sound when unread counts increase, based on user preferences. [1] [2]

Default preferences update:

  • Changes the default inbox cue preferences so that sound notifications are enabled by default, and updates tests accordingly. [1] [2]

Testing improvements:

  • Adds a test to ensure that broadcastInboxCuePreferencesUpdated correctly dispatches the custom event for same-tab listeners.

@vercel
Copy link

vercel bot commented Mar 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
opencom-landing Ready Ready Preview, Comment Mar 4, 2026 4:44pm
opencom-web Ready Ready Preview, Comment Mar 4, 2026 4:44pm
opencom-widget Ready Ready Preview, Comment Mar 4, 2026 4:44pm

@djanogly djanogly merged commit c3ff492 into main Mar 4, 2026
5 of 6 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