Automattic / breakingbot

breaking incident management chat bot
GNU General Public License v2.0
1 stars 0 forks source link

Seperate notification threshold from high/low priority dynamic #63

Open mhsdef opened 1 month ago

mhsdef commented 1 month ago

We have a desired use case of:

p1 - alert loud to notify channel p2 (.start) - high priority, but no notify p3 (.low) - low priority, no notify ...

Right now, notify is hinged on start/low. Instead, we can explicitly put a priority threshold into configs.

### Tasks
- [ ] Add a notify priority level config key/value
- [ ] Adjust create, notify, resolve logic in `handlers` to leverage it