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

Add the ability to emit platform wide notifications #4637

Merged
merged 24 commits into from
Nov 6, 2024
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
bb2f11d
Add a new notifications hub admin page component and related menu entry
cstns Oct 9, 2024
b71828d
frontend api call scaffolding
cstns Oct 10, 2024
4880c35
Merge remote-tracking branch 'origin/main' into platform-wide-notific…
cstns Oct 11, 2024
d9ebc97
naive implementation of backend API
cstns Oct 11, 2024
c2f43d8
add admin ui to send an announcement
cstns Oct 11, 2024
797c566
add the ability to send a mock announcement request and form validati…
cstns Oct 11, 2024
02dd06c
change text key to message to comply with notification payload
cstns Oct 11, 2024
a4aed86
Merge remote-tracking branch 'origin/main' into platform-wide-notific…
cstns Oct 14, 2024
59dd18d
add url to notification
cstns Oct 14, 2024
865149f
add url params to notifications backend
cstns Oct 14, 2024
a7dcc0a
reformat
cstns Oct 14, 2024
6aeb4f0
fix the byTeamRole query to include admins
cstns Oct 14, 2024
8df4115
prevent mark notification as read api call if notification already read
cstns Oct 14, 2024
e67d4c1
qf instance crash notification not redirecting
cstns Oct 14, 2024
6aa276a
Merge remote-tracking branch 'origin/main' into platform-wide-notific…
cstns Oct 14, 2024
25665a5
fix admin menu entry data attr
cstns Oct 14, 2024
14d7cb1
add e2e tests for the admin notification-hub + locators
cstns Oct 14, 2024
14cf1b1
add e2e tests for the notification pill
cstns Oct 14, 2024
a5c9e04
remove the external url selector forcing external url for platform wi…
cstns Oct 21, 2024
fcf5bbf
Merge branch 'main' into platform-wide-notifications
cstns Oct 24, 2024
cfd9c50
Refactor to send notifications in batches of 200
knolleary Nov 5, 2024
cd4b824
Cosmetic improvement on the notification role selection
knolleary Nov 5, 2024
949de2f
Fix ui tests following cosmetic change
knolleary Nov 5, 2024
4ea0309
Add unit tests for admin notifications api
knolleary Nov 6, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'origin/main' into platform-wide-notific…
…ations
cstns committed Oct 14, 2024
commit 6aa276a1e045b7fd0fbcaedec0f672086ac3bdd3

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.