Page MenuHomePhabricator

Don't notify without notifiable attendees
ClosedPublic

Authored by 0 on Dec 21 2017, 8:44 PM.
Tags
None
Referenced Files
F15456144: D18835.id45191.diff
Sun, Mar 30, 7:19 AM
F15429240: D18835.id45191.diff
Mon, Mar 24, 1:17 AM
F15409528: D18835.id45191.diff
Wed, Mar 19, 4:39 AM
F15382873: D18835.diff
Fri, Mar 14, 2:48 PM
F15331946: D18835.id45191.diff
Fri, Mar 7, 3:33 PM
F15302795: D18835.id45190.diff
Wed, Mar 5, 9:52 PM
Unknown Object (File)
Mar 1 2025, 8:26 PM
Unknown Object (File)
Feb 22 2025, 3:46 PM
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