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.

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

Notification events should be extensible (not just email) #29

Closed
jathanism opened this Issue Aug 8, 2012 · 1 comment

Comments

Projects
None yet
1 participant
@jathanism
Member

jathanism commented Aug 8, 2012

This will require adding a hook to the Trigger configuration for when alerts are to be sent. Currently it's assumed that an email will be sent, but this will need to be changed to just perform a function call. The default function call will just send email as it does now. This is similiar to the current implementation of get_current_oncall() and create_cm_ticket() from within the Trigger settings.py.

@ghost ghost assigned jathanism Sep 11, 2012

@jathanism

This comment has been minimized.

Member

jathanism commented Sep 25, 2012

It looks like we should change this whole method do use a handler pattern. Stay tuned.

@jathanism jathanism closed this Oct 3, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment