Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Add ability to turn off daily reports #3664

Merged
whitdog47 merged 8 commits into from
Aug 8, 2023
Merged

Add ability to turn off daily reports #3664

whitdog47 merged 8 commits into from
Aug 8, 2023

Conversation

@whitdog47
Copy link
Contributor

@whitdog47 whitdog47 commented Aug 5, 2023 •
edited
Loading

This PR adds the ability for each project to turn off the sending of daily incident reports.

@whitdog47 whitdog47 added the enhancement New feature or request label Aug 5, 2023
@whitdog47 whitdog47 requested a review from wssheldon August 5, 2023 00:16
@kevgliss
Copy link
Contributor

kevgliss commented Aug 7, 2023

Have you thought about adding this setting to the Notification model instead? That would provide more granularity over which destinations receive which messages.

@kevgliss kevgliss self-requested a review August 8, 2023 22:20
@whitdog47 whitdog47 merged commit 7288f74 into master Aug 8, 2023
@whitdog47 whitdog47 deleted the enhancement/add-ability-to-turn-off-daily-reports branch August 8, 2023 23:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@wssheldon wssheldon Awaiting requested review from wssheldon

1 more reviewer

@kevgliss kevgliss kevgliss approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@whitdog47 @kevgliss