search

PagerNation / Escalator-API

MIT License
3 stars 0 forks source link

Remove EP and add delays #38

Closed jarrydlee closed 7 years ago

jarrydlee commented 7 years ago

@PagerNation/all we spoke about this change a little bit, this is just a quick attempt I threw together on the plane.

It removes the EP on user documents because it's redundant when all devices will be on the EP. This will also allow us to rearrange devices in a users EP without rearranging the delay in between (something Justin wanted for usability)

kalebingerick commented 7 years ago

So instead of an escalation policy with devices and delays, we have a list of delays and a list of devices separately?

jarrydlee commented 7 years ago

Yes, the idea is that we need to be able to keep an ordering on the devices and an ordering on the delays as they should be updated separately

jarrydlee commented 7 years ago

Merging this for now to move forward with other work. I don't think the solution is perfect, but I also don't think it's worth getting hung up on it right now