Open
Description
openedon Sep 4, 2024
Feature Description
This issue should refactor CoreSiteBannerNotifications
so that it uses the new datastore infrastructure to register and queue the notification. It should also incorporate a new "layout".
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
- The
CoreSiteBannerNotifications
component should be refactored so that it is registered and rendered (queued) using the newcore/notifications
datastore. - This notification component should not be called directly (i.e. in
BannerNotifications
) but only via the genericgetQueuedNotifications
selector. - The refactored component should not contain any business logic that hides it / prevents it from rendering. This logic should be contained in a callback function defined during registration.
- The component should not use the bloated
BannerNotification
component. Instead, it should be rendered using the newNotification
component wrapper and a simpler "layout" component that solely encapsulates its structure and design.
Implementation Brief
Test Coverage
QA Brief
Changelog entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment