HomePhabricator

Reload subscribers before sending mail, etc

Description

Reload subscribers before sending mail, etc

Summary: Fixes T4629. CCs added by Herald don't get added to the cached subscriber list. Just reload subscribers before sending mail to pick up effects.

Test Plan: Created an "always add X as CC" Herald rule for revisions, created a revision, saw them get initial mail.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: spicyj, epriestley

Maniphest Tasks: T4629

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Mar 18 2014, 11:01 PM
Reviewer
btrahan
Differential Revision
D8565: Reload subscribers before sending mail, etc
Parents
rP5721560663de: Use modern UI for OAuthServer details page
Branches
Unknown
Tags
Unknown
Tasks
T4629: Herald: Not receiving email on diff creation

Event Timeline