Skip to content

Notifications System Core#54

Merged
ChrisPenner merged 32 commits into
mainfrom
cp/notifications-core
Apr 14, 2025
Merged

Notifications System Core#54
ChrisPenner merged 32 commits into
mainfrom
cp/notifications-core

Conversation

@ChrisPenner
Copy link
Copy Markdown
Member

@ChrisPenner ChrisPenner commented Apr 8, 2025

Overview

TODO:

  • Add checks to ensure the subscriber has access to subscribe to notifications on a given resource/thing Changed it to do a check before creating the notification rather than before creating subscription.
  • Record notification events on contribution create and branch update events
  • Add bulk notification status event update APIs.

Adds the core tables and APIs for managing the notifications system.

MVP supports only a small subset of what's to come.

MVP:

  • allows subscribing to contribution creation events within a user or project
  • allows subscribing to branch update events within a user or project
  • provides APIs to back a Notifications Hub
  • includes the backbone for email and webhook notifications, but they're not wired in yet.

@ChrisPenner ChrisPenner marked this pull request as ready for review April 10, 2025 19:43
@ChrisPenner ChrisPenner force-pushed the cp/notifications-core branch from f4ef8f9 to 9a0b42b Compare April 10, 2025 19:44
@ChrisPenner ChrisPenner changed the base branch from main to bump-unison April 10, 2025 20:06
@ChrisPenner ChrisPenner requested a review from a team as a code owner April 10, 2025 20:06
@ChrisPenner ChrisPenner changed the base branch from bump-unison to main April 10, 2025 20:06
@ChrisPenner ChrisPenner force-pushed the cp/notifications-core branch from 50412d0 to 626fe11 Compare April 10, 2025 20:07
@ChrisPenner ChrisPenner merged commit 012b5cb into main Apr 14, 2025
8 checks passed
@ChrisPenner ChrisPenner deleted the cp/notifications-core branch April 14, 2025 15:07
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.

1 participant