Skip to content

chore: remove notifications experiment #14869

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 8 commits into from
Oct 1, 2024
Merged

chore: remove notifications experiment #14869

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5cc1e73
chore: remove notifications experiment
dannykopping Sep 30, 2024
cb0be34
Pass transaction into report generator to avoid deadlock
dannykopping Sep 30, 2024
6e7c919
Only close done channel when notifier is present
dannykopping Sep 30, 2024
7c571ae
Reorder notification manager to avoid race with CreateInMemoryTaggedP…
dannykopping Sep 30, 2024
0107ad1
Remove experiment reference & show beta badge in frontend
dannykopping Oct 1, 2024
0aa6d7b
Ensuring done channel cannot be closed twice
dannykopping Oct 1, 2024
bf1a148
Beta tag on account notification settings pages
dannykopping Oct 1, 2024
27d28b3
make fmt
dannykopping Oct 1, 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
Prev Previous commit
Next Next commit
Only close done channel when notifier is present 
Signed-off-by: Danny Kopping <danny@coder.com>
  • Loading branch information
@dannykopping
dannykopping committed Oct 1, 2024
commit 6e7c919c79826687f7fbcda3c8497b263726c17c
4 changes: 3 additions & 1 deletion coderd/notifications/manager.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,9 @@ func (m *Manager) Run(ctx context.Context) {
// events, creating a notifier, and publishing bulk dispatch result updates to the store.
func (m *Manager) loop(ctx context.Context) error {
defer func() {
close(m.done)
if m.notifier != nil {
close(m.done)
}
m.log.Info(context.Background(), "notification manager stopped")
}()

Expand Down