Skip to content

Conversation

@pd-redis
Copy link
Collaborator

This pull request refactors the notifications system in the UI to improve modularity, maintainability, and user experience. The main changes include splitting notification logic into custom hooks, introducing a dedicated component for infinite notifications, and updating toast handling to support multiple containers and better queue management.

Notifications system refactor:

  • Extracted notification logic into custom hooks: useErrorNotifications, useMessageNotifications, and useInfiniteNotifications, improving separation of concerns and code readability.

Infinite notifications improvements:

  • Added a new InfiniteNotifications component to handle infinite notifications with a queue and timed dismissals, using a dedicated container.
  • The implementation is still not up to par with EUI implementation since redis-ui component lacks the ability to update contents of the toast, which EUI does by default, therefore pushing repeating notifications is visible for the user by removing and adding back the toast.
  • potential further improvement would be to compare new notifications with the one that is currently rendered and skip it if it is the same

@github-actions
Copy link
Contributor

github-actions bot commented Oct 28, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.17% 20167/24542
🟡 Branches 67.55% 8723/12913
🟡 Functions 76.35% 5364/7026
🟢 Lines 82.59% 19746/23909

Test suite run success

5226 tests passing in 681 suites.

Report generated by 🧪jest coverage report action from 4c64b06

@pd-redis pd-redis mentioned this pull request Oct 29, 2025
@pd-redis pd-redis self-assigned this Oct 29, 2025
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.

4 participants