Skip to content

Conversation

@rafaeltonholo
Copy link
Member

Changeset split from #9415
Part of #9245.

  • Improve the Notification API surface
    • Flatten MailNotification.NewMail subclasses
    • Moved SystemNotification-related properties from base Notification interface to SystemNotification interface
    • Flatten CommandOutcome subclasses
    • Removed lockscreenNotification property in favour of asLockscreenNotification method.

Copy link
Member

@wmontwe wmontwe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@rafaeltonholo rafaeltonholo force-pushed the feat/9245/notification-type-api branch from ca81e33 to 947a65f Compare July 24, 2025 12:51
@rafaeltonholo rafaeltonholo force-pushed the feat/9245/notification-type-api branch from 947a65f to 83757b0 Compare July 24, 2025 13:26
@rafaeltonholo rafaeltonholo merged commit d812a31 into thunderbird:main Jul 24, 2025
3 checks passed
@rafaeltonholo rafaeltonholo deleted the feat/9245/notification-type-api branch July 24, 2025 14:40
@thunderbird-botmobile thunderbird-botmobile bot added this to the Thunderbird 13 milestone Jul 24, 2025
@rafaeltonholo rafaeltonholo linked an issue Jul 29, 2025 that may be closed by this pull request
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.

In-App Error Notifications - Create NotificationSender

2 participants