Skip to content

Conversation

@ekraffmiller
Copy link
Contributor

@ekraffmiller ekraffmiller commented Sep 9, 2025

What this PR does / why we need it:

Implements the Notifications tab of the Account Page.

Which issue(s) this PR closes:

Special notes for your reviewer:

This implementation uses a NotificationContext, and polling to get notification updates. I didn't add the icons next to the messages because I don't think they add much value, but if it's wanted I can add them.

Note that a

Suggestions on how to test this:

Do some actions in the SPA to trigger notifications. For example, create a new account, publish a dataset, upload a .csv file, or assign a role. The actions should update the Header to show the new unread notifications. Go to the Notifications tab to view them.

Note that a follow up PR will introduce paging, so that will handle the performance issues currently seen in JSF when there is a large amount of notifications.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Yes, rather than showing new notifications highlighted until the page is refreshed, as done in JSF, the SPA version of the page removes the highlight of the new notifications after a 2 second delay. After two seconds, the new notifications are marked as read, and the highlights are removed, and also the unread notifications badge is removed from the Header.

JSF Version

Screenshot 2025-09-24 at 2 39 29 PM

SPA Version

Screenshot 2025-09-24 at 2 38 49 PM

Is there a release notes update needed for this change?:

Additional documentation:

@coveralls
Copy link

coveralls commented Sep 9, 2025

Coverage Status

coverage: 98.249% (+0.4%) from 97.83%
when pulling 7493817 on 775-account-page-notifications
into 212fa23 on develop.

@cmbz cmbz added the FY26 Sprint 8 FY26 Sprint 8 (2025-10-08 - 2025-10-22) label Oct 8, 2025
@g-saracca g-saracca removed their assignment Oct 15, 2025
@g-saracca g-saracca moved this from In Review 🔎 to In Progress 💻 in IQSS Dataverse Project Oct 15, 2025
@ekraffmiller ekraffmiller moved this from In Progress 💻 to Ready for Review ⏩ in IQSS Dataverse Project Oct 20, 2025
@ekraffmiller ekraffmiller removed their assignment Oct 21, 2025
@cmbz cmbz added the FY26 Sprint 9 FY26 Sprint 9 (2025-10-22 - 2025-11-05) label Oct 23, 2025
@g-saracca g-saracca moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Oct 27, 2025
@g-saracca g-saracca self-assigned this Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY26 Sprint 7 FY26 Sprint 7 (2025-09-24 - 2025-10-08) FY26 Sprint 8 FY26 Sprint 8 (2025-10-08 - 2025-10-22) FY26 Sprint 9 FY26 Sprint 9 (2025-10-22 - 2025-11-05) GREI Re-arch GREI re-architecture-related Original size: 3 Size: 3 A percentage of a sprint. 2.1 hours. SPA.Q3.2025.6 Account Page: Notifications

Projects

Status: In Review 🔎

Development

Successfully merging this pull request may close these issues.

Account Page: Notifications - UI Integration

4 participants