Page MenuHomePhabricator

Don't notify without notifiable attendees
ClosedPublic

Authored by 0 on Dec 21 2017, 8:44 PM.
Tags
None
Referenced Files
F15575733: D18835.id45190.diff
Tue, May 6, 6:35 AM
F15538993: D18835.diff
Thu, Apr 24, 10:44 PM
F15526497: D18835.diff
Mon, Apr 21, 9:09 PM
F15525788: D18835.id45191.diff
Mon, Apr 21, 4:53 PM
F15519427: D18835.id.diff
Sat, Apr 19, 10:55 PM
F15514645: D18835.diff
Fri, Apr 18, 4:12 AM
F15496799: D18835.diff
Sun, Apr 13, 8:07 AM
F15456144: D18835.id45191.diff
Mar 30 2025, 7:19 AM
Subscribers

Details

Summary

Events with no attendees (e.g. fresh instances of recurring events) would trigger an exception when sending notifications, because $attendee_map would still get populated.

Test Plan

Declined event, restarted daemons. Did not see exception. Accepted event, restarted daemons. Saw "[Calendar] [Reminder]" email.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable