Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitd6d723f

Browse files
defelmnqEdwardAngert
authored andcommitted
feat(coderd): add inbox notifications endpoints (#16889)
This PR is part of the inbox notifications topic, and rely on previousPRs merged - it adds :- Endpoints to : - WS : watch new inbox notifications - REST : list inbox notifications - REST : update the read status of a notificationAlso, this PR acts as a follow-up PR from previous work and : - fix DB query issues- fix DBMem logic to match DB
1 parentc83e53b commitd6d723f

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
iferr!=nil {
939939
returnxerrors.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 aboutcustomizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp