Skip to content

fix(notifications): use megaphone icon for major headline notifications#6017

Merged
nimrodkra merged 1 commit into
mainfrom
fix/major-headline-notification-icon
May 10, 2026
Merged

fix(notifications): use megaphone icon for major headline notifications#6017
nimrodkra merged 1 commit into
mainfrom
fix/major-headline-notification-icon

Conversation

@nimrodkra
Copy link
Copy Markdown
Member

@nimrodkra nimrodkra commented May 10, 2026

Summary

Follow-up to #5922:

Adds NotificationIconType.Megaphone and maps it to the existing MegaphoneIcon so the in-app major headline notification matches the Happening Now sidebar entry / Headlines mobile navbar icon. Pairs with the API change adding NotificationIcon.Megaphone.

Test plan

  • pnpm --filter shared lint
  • Verify a Breaking major headline notification renders with the megaphone icon in the notification panel

Made with Cursor

Preview domain

https://fix-major-headline-notification.preview.app.daily.dev

Maps the new NotificationIconType.Megaphone to MegaphoneIcon so the in-app
major headline notification matches the Happening Now sidebar / Headlines
mobile navbar icon.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 2026

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview May 10, 2026 1:17pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored May 10, 2026 1:17pm

Request Review

@nimrodkra nimrodkra merged commit 8c7b956 into main May 10, 2026
14 checks passed
@nimrodkra nimrodkra deleted the fix/major-headline-notification-icon branch May 10, 2026 13:20
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.

2 participants