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

feat: add killswitch for notifications#13794

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mtojek merged 22 commits intomainfrom4-pause-1
Jul 10, 2024
Merged
Changes from1 commit
Commits
Show all changes
22 commits
Select commitHold shift + click to select a range
735d0ef
feat: add killswitch for notifications
mtojekJul 9, 2024
b6338e5
revert
mtojekJul 10, 2024
16c8950
migrate down
mtojekJul 10, 2024
edfff5e
304
mtojekJul 10, 2024
89e1396
Notifications settings
mtojekJul 10, 2024
ab8a6c7
debug message
mtojekJul 10, 2024
2522366
return true
mtojekJul 10, 2024
7dfbd29
remove WillUsePostgres
mtojekJul 10, 2024
d7f470d
Merge branch 'main' into 4-pause-1
mtojekJul 10, 2024
6abf39f
fix client 304
mtojekJul 10, 2024
341485f
createSampleUser
mtojekJul 10, 2024
c90ff21
sdkError
mtojekJul 10, 2024
da6775f
remove test for not modified
mtojekJul 10, 2024
9d430bb
swagger update
mtojekJul 10, 2024
a55fc8a
bring back Settings not modified
mtojekJul 10, 2024
af36afb
enable WillUsePostgres in TestWebhookDispatch
mtojekJul 10, 2024
7e8ad09
refactor Settings not modified
mtojekJul 10, 2024
ce1a9e9
refactored TestNotifierPaused
mtojekJul 10, 2024
dbf6a40
bug: RLock RUnlock
mtojekJul 10, 2024
c2e181c
Danny's review
mtojekJul 10, 2024
fa0f381
Merge branch 'main' into 4-pause-1
mtojekJul 10, 2024
83978e7
fix
mtojekJul 10, 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
PrevPrevious commit
NextNext commit
bring back Settings not modified
  • Loading branch information
@mtojek
mtojek committedJul 10, 2024
commita55fc8a8ad55e6d5c66f0f84784c633a814cb3da
18 changes: 18 additions & 0 deletionscoderd/notifications_test.go
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -60,4 +60,22 @@ func TestUpdateNotificationsSettings(t *testing.T) {
require.NoError(t, err)
require.Equal(t, expected, actual)
})

t.Run("Settings not modified", func(t *testing.T) {
t.Parallel()

client := coderdtest.New(t, nil)
_ = coderdtest.CreateFirstUser(t, client)

ctx := testutil.Context(t, testutil.WaitShort)

// given
expected := codersdk.NotificationsSettings{
NotifierPaused: false,
}

// then
err := client.PutNotificationsSettings(ctx, expected)
require.NoError(t, err)
})
}

[8]ページ先頭

©2009-2025 Movatter.jp