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 sub ld #1559

Merged
merged 211 commits into from
Dec 13, 2024
Merged

merge sub ld #1559

merged 211 commits into from
Dec 13, 2024

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 <bflynn@luckyrabbit.tech>
@Dark-Knight-1313 Dark-Knight-1313 merged commit 98474e1 into master Dec 13, 2024
84 of 102 checks passed
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.

6 participants