coder / coder/coder

feat: allow pausing enqueueing of notifications

Open
#14,373 0 comments 3 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
14.5k
Forks
1.5k
Avg merge
1d 20h
Merged PRs (30d)
601

Description

Notifications are currently enqueued (inserted into database) and then dispatched (email sent, webhook POSTed). We currently allow disabling the dispatch of notifications only. However, there are scenarios where it may be necessary to stop enqueuing notifications temporarily.

For example, if an administrator is about to perform a manual action that will cause a significant number of notifications to be enqueued and potentially rack up a large bill for a 3rd-party service, they may decide to pre-emptively disable enqueueing new notifications before taking this action.

Add a separate `EnqueuerPaused` boolean flag to `NotificationsSettings` to allow administrators to temporarily disable the enqueuing of notifications.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.