Skip to content
This repository has been archived by the owner on Aug 18, 2023. It is now read-only.

Implement support for easily determining what notifiers are enabled #29

Open
atc0005 opened this issue Apr 22, 2020 · 0 comments
Open
Assignees
Milestone

Comments

@atc0005
Copy link
Owner

atc0005 commented Apr 22, 2020

While implementing #21, I've setup a NotifyTeams() and NotifyEmail() pair of methods to determine whether those notification types are enabled. Unfortunately they're tied directly to a Config type, so only the value created from that type can be used to call the methods. This either means I pass the Config value to each part of the code that needs it, or I find a way to track and expose the state of all notifiers (enabled/disabled) from relevant parts of the code that need it.

For now, the stats output related to Teams (#21) and Email notifications (#19) will be generated regardless of whether they're actually used.

@atc0005 atc0005 added this to the Future milestone Apr 22, 2020
@atc0005 atc0005 self-assigned this Apr 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant