Homestyx hydra
Diffusion hydra 393824656fcc

Don't notify without notifiable attendees

Description

Don't notify without notifiable attendees

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.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D18835

Details

Provenance
Dmitri IouchtchenkoAuthored on Dec 21 2017, 3:46 PM
epriestleyCommitted on Dec 21 2017, 3:46 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:e411d75964e5: Fix an issue where blame could fatal for unrecognized authors
Branches
Unknown
Tags
Unknown

Event Timeline