Skip to content

Commit 3ae55bb

Browse files
defelmnqdefelmnq
authored
feat(coderd): add inbox notifications endpoints (coder#16889)
This PR is part of the inbox notifications topic, and rely on previous PRs merged - it adds : - Endpoints to : - WS : watch new inbox notifications - REST : list inbox notifications - REST : update the read status of a notification Also, this PR acts as a follow-up PR from previous work and : - fix DB query issues - fix DBMem logic to match DB
1 parent e85c92e commit 3ae55bb

20 files changed

+2091
-63
lines changed

‎cli/server.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -934,7 +934,7 @@ func (r *RootCmd) Server(newAPI func(context.Context, *coderd.Options) (*coderd.
934934
// The notification manager is responsible for:
935935
// - creating notifiers and managing their lifecycles (notifiers are responsible for dequeueing/sending notifications)
936936
// - keeping the store updated with status updates
937-
notificationsManager, err = notifications.NewManager(notificationsCfg, options.Database, helpers, metrics, logger.Named("notifications.manager"))
937+
notificationsManager, err = notifications.NewManager(notificationsCfg, options.Database, options.Pubsub, helpers, metrics, logger.Named("notifications.manager"))
938938
if err != nil {
939939
return xerrors.Errorf("failed to instantiate notification manager: %w", err)
940940
}

‎coderd/apidoc/docs.go

Lines changed: 206 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)