Skip to content

feat(coderd): add mark-all-as-read endpoint for inbox notifications #16976

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Mar 20, 2025
Merged

feat(coderd): add mark-all-as-read endpoint for inbox notifications #16976

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
8d3ca0a
work on inbox notifications mark-all-as-read endpoint
defelmnq Mar 18, 2025
e4e2967
fix linter
defelmnq Mar 18, 2025
dd8ee20
fix gen
defelmnq Mar 18, 2025
8aaace3
fix dbauthz
defelmnq Mar 18, 2025
0d98bf6
improve dbauthz
defelmnq Mar 18, 2025
27ad10b
make fmt
defelmnq Mar 18, 2025
452e92b
change endpoint status code and add some more details
defelmnq Mar 19, 2025
deff124
fix 204
defelmnq Mar 19, 2025
0815e96
work on swagger
defelmnq Mar 19, 2025
a1567a0
work on swagger
defelmnq Mar 19, 2025
7881887
work on swagger
defelmnq Mar 19, 2025
9d871fc
fixes from review
defelmnq Mar 20, 2025
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
fix 204
  • Loading branch information
@defelmnq
defelmnq committed Mar 19, 2025
commit deff124bf1a926a24af221ac6fb1b4d6d17e32a9
3 changes: 1 addition & 2 deletions coderd/inboxnotifications_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -771,9 +771,8 @@ func TestInboxNotifications_MarkAllAsRead(t *testing.T) {
require.Equal(t, 20, notifs.UnreadCount)
require.Len(t, notifs.Notifications, 20)

resp, err := client.MarkAllInboxNotificationsAsRead(ctx)
err = client.MarkAllInboxNotificationsAsRead(ctx)
require.NoError(t, err)
require.Zero(t, resp.UnreadCount)

notifs, err = client.ListInboxNotifications(ctx, codersdk.ListInboxNotificationsRequest{})
require.NoError(t, err)
Expand Down
11 changes: 5 additions & 6 deletions codersdk/inboxnotification.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,21 +114,20 @@ type MarkAllInboxNotificationsAsReadResponse struct {
UnreadCount int `json:"unread_count"`
}

func (c *Client) MarkAllInboxNotificationsAsRead(ctx context.Context) (MarkAllInboxNotificationsAsReadResponse, error) {
func (c *Client) MarkAllInboxNotificationsAsRead(ctx context.Context) error {
res, err := c.Request(
ctx, http.MethodPut,
"/api/v2/notifications/inbox/mark-all-as-read",
nil,
)
if err != nil {
return MarkAllInboxNotificationsAsReadResponse{}, err
return err
}
defer res.Body.Close()

if res.StatusCode != http.StatusOK {
return MarkAllInboxNotificationsAsReadResponse{}, ReadBodyAsError(res)
if res.StatusCode != http.StatusNoContent {
return ReadBodyAsError(res)
}

var resp MarkAllInboxNotificationsAsReadResponse
return resp, json.NewDecoder(res.Body).Decode(&resp)
return nil
}
Loading