Page MenuHomePhabricator

Differential drops draft mail when prototypes are enabled but no build plans are configured
Closed, ResolvedPublic

Description

See PHI271. If you have prototypes enabled but do not use Harbormaster to apply builds, initial "Review Requested" mail can be dropped.

Specifically, after D18819, transactions are split into two groups by the Herald passes. With the specified configuration, the second group contains only a "Status: Needs Review" transaction. This transaction is marked as a create transaction, so it is hidden by default. Since all transactions in the group are hidden, mail does not try to generate, and we never reach the stage later on where we fill in all the other transactions.

(If Harbormaster is configured to do things, the group includes a "builds passed" transaction and isn't a create transaction. If prototypes are off, transactions don't get split into two groups.)

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley triaged this task as Normal priority.Jan 3 2018, 9:40 PM
epriestley created this task.
epriestley added a revision: Restricted Differential Revision.Jan 3 2018, 9:43 PM
epriestley added a revision: Restricted Differential Revision.Jan 3 2018, 9:58 PM
epriestley added a commit: Restricted Diffusion Commit.Jan 4 2018, 4:14 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.
epriestley claimed this task.

The install in PHI271 reported this as fixed after upgrading.