scobal/seyren — pull request #62 by EmmanuelAcheampong (2012-10-25 12:17:48+00:00)
Title: Adding PagerDuty Notifications

this makes use of biz.neustar PagerDuty work (https://github.com/webmetrics/pagerduty-java).

Its not perftect, but that's due to the model rather than the work i've added, for some reason the last change introduced to the main repo added a new method to the NotificationService.java interface all other notifications are supposedly meant to implement, question why you would have a sendemailstatus method you want to override in all other types of notification services?? but for now, just going to get in the changes, also need to look at the whole structure as the current model doesn't really work for anything more than one notification type of email, unless my Java can be improved and we can somehow infer at runtime which notification type to fire off based on the notification type of the subscription being checked.
any improvements/comments are welcome.
Will need to update the readme to include a note about adding a PAGERDUTY_DOMAIN env variable if the use should want to use this.