If I understand correctly, any event definition with SendSMS as the reactor and HoldIsAvailable, HoldIsCancelled, or HoldNotifyCheck as the validator should have a check_sms_notify parameter. I wonder if we should add an upgrade script that adds the param, if it's missing, to all event definitions (including custom ones) that meet those criteria.
It looks like Sitka is affected by this issue.
If I understand correctly, any event definition with SendSMS as the reactor and HoldIsAvailable, HoldIsCancelled, or HoldNotifyCheck as the validator should have a check_sms_notify parameter. I wonder if we should add an upgrade script that adds the param, if it's missing, to all event definitions (including custom ones) that meet those criteria.