Skip to content

APPT-1901 - Notification banner v10 updates#1495

Open
ste-banks wants to merge 4 commits intomainfrom
appt-1901/notification-banner-v10
Open

APPT-1901 - Notification banner v10 updates#1495
ste-banks wants to merge 4 commits intomainfrom
appt-1901/notification-banner-v10

Conversation

@ste-banks
Copy link
Contributor

@ste-banks ste-banks commented Mar 9, 2026

Description

Notification banner updated to use react components. The close action on the notification banner has been removed as it's not part of the design system and after speaking with Jon we decided we'd remove it. If it gets added back into the design system we will add that functionality back in but for now, the react components version of the notification banner doesn't know the concept of closing.

They banner itself is now scoped to a path so if a user navigates away from from the page the banner will disappear but if they navigate back within the default timeout it will reappear. The notification banner will still disappear after 15 seconds when performing an action as per original functionality.

image image image image image image

Fixes # (issue)

Checklist:

  • My work is behind a feature toggle (if appropriate)
  • If my work is behind a feature toggle, I've added a full suite of tests for both the ON and OFF state
  • The ticket number is in the Pull Request title, with format "APPT-XXX: My Title Here"
  • I have ran npm tsc / lint (in the future these will be ran automatically)
  • My code generates no new .NET warnings (in the future these will be treated as errors)
  • If I've added a new Function, it is disabled in all but one of the terraform groups (e.g. http_functions)
  • If I've added a new Function, it has both unit and integration tests. Any request body validators have unit tests also
  • If I've made UI changes, I've added appropriate Playwright and Jest tests
  • If I've added/updated an end-point, I've added the appropriate annotations and tested the Swagger documentation reflects the change

@ste-banks ste-banks changed the title Notification banner v10 updates - WIP APPT-1901 - Notification banner v10 updates Mar 10, 2026
@ste-banks ste-banks marked this pull request as ready for review March 10, 2026 13:55
@sonarqubecloud
Copy link

Copy link
Contributor

@pata9 pata9 left a comment

Choose a reason for hiding this comment

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

Guessing you can't dismiss/close the notification anymore? Have we ran that past the design guys?

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.

3 participants