Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge LD sub sub #1557

Closed
wants to merge 116 commits into from
Closed

Merge LD sub sub #1557

wants to merge 116 commits into from

Conversation

Dark-Knight-1313
Copy link
Collaborator

Story: https://qmacbis.atlassian.net/browse/OY2-XXXX
Endpoint: See github-actions bot comment

Details

A quick summary of the intended outcome of this PR.

Changes

  • Specific functionality changes, or
  • A specific bug addressed in this PR

Implementation Notes

  • Developer focused info that may affect future work

Test Plan

  1. Test step 1
  2. Test step 2

anyoussefinia and others added 30 commits September 9, 2024 14:12
… Application (#1512)

* started base UI componets for banner & card, not tested, still need to style

* first go on system notifications

* first shot at user notifications

* dismissUserNotification api + unit tests

* adding api for creating user notifications

* lint fix

* return created items when creating user notifications

* updating styling to match mockups, added tests for components (#1504)

* remove double layer of status code and body

* make create pass id (email) as a url token

* added the notificationAPI, added test, updated components slightly to work with be,  added user nots to context

* added local storage so notification stays on refresh, remove storage on logout:)

* dismiss existing notifications upon login

---------

Co-authored-by: Brandon Flynn <[email protected]>
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.

5 participants