Comment 5 for bug 1096209

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

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.