ui/sched: always use Dest API

target / goalert

Open source on-call scheduling, automated escalations, and notifications so you never miss a critical alert

https://goalert.me

Apache License 2.0

2.22k stars 240 forks source link

ui/sched: always use Dest API #3785

Closed mastercactapus closed 4 months ago

mastercactapus commented 6 months ago

Description: Updates schedule on-call notifications always to use the new dest API.

Which issue(s) this PR fixes: Part of #2639

Describe any introduced user-facing changes: N/A

Describe any introduced API changes: N/A

Additional Info:

Fixed issue with dest dialogs where incorrect times were recorded when sched. TZ was different than local

mastercactapus commented 6 months ago

Waiting for #3779 to merge

mastercactapus commented 4 months ago

@Forfold It's existing behavior untouched by this work. I agree it should be changed but maybe do that in it's own PR since there may be a bit of refactoring and dialog updates to handle no valid types.

Forfold commented 4 months ago

Gotcha. As long as we don't lose track of it..