hubot-archive / hubot-pager-me

PagerDuty integration for Hubot
https://www.npmjs.com/package/hubot-pager-me
MIT License
73 stars 92 forks source link

Fix /pager trigger when user has email address, but no PagerDuty user found for it #22

Closed technicalpickles closed 9 years ago

technicalpickles commented 9 years ago

This is a followup to https://github.com/hubot-scripts/hubot-pager-me/pull/20 . That addressed doing /pager me trigger when your user doesn't have any email associated with it (like using campfire shell). If you are using Campfire and some other adapters, it defaults to your user's email address on the chat service, which might not be your correct address. When a PagerDuty user isn't found for that email address, /pager trigger failed.

This applies the not required logic for when there is a configured email address, but it doesn't result in any found pagerduty users.