search

stripe-contrib / pagerbot

Manage Pagerduty on-call schedules from within your internal chat channels.
https://stripe.com/blog/pagerbot
MIT License
277 stars 90 forks source link

Add emails as an index for pagerbot users. #40

Closed kiran closed 8 years ago

kiran commented 8 years ago

Pagerbot, by default, only indexes on PagerDuty IDs and aliases. Since Pagerduty emails are likely to be at least somewhat unique, add those in as a default index, too.

r? @rhwlo

rhwlo commented 8 years ago

Looks good to me, though per our discussion, we should check if the key already exists and log if it does, just for the weird corner cases.