Page MenuHomePhabricator

Prepare revision mail for the "Draft" status
ClosedPublic

Authored by epriestley on Sep 18 2017, 9:48 PM.
Tags
None
Referenced Files
F18580157: D18627.diff
Thu, Sep 11, 2:37 AM
F18527432: D18627.diff
Sat, Sep 6, 6:49 PM
F18527431: D18627.diff
Sat, Sep 6, 6:49 PM
F18527409: D18627.diff
Sat, Sep 6, 6:45 PM
F18509119: D18627.id.diff
Fri, Sep 5, 3:14 AM
F18501660: D18627.diff
Thu, Sep 4, 9:53 PM
F18208866: D18627.id.diff
Mon, Aug 18, 9:58 PM
F18193726: D18627.diff
Sun, Aug 17, 6:31 AM
Subscribers
None

Details

Summary

Ref T2543. Currently, we always do some special things when a revision is created, mostly adding more stuff to the mail.

With drafts, we want to suppress initial mail and send this big, rich mail only when the revision actually moves out of "draft".

Prepare the code for this, with the actual methods hard-coded to the current behavior. This will probably take some tweaking but I think I got most of it.

Test Plan

Banged around in Differential so it sent some mail, saw normal mail without anything new.

Diff Detail

Repository
rP Phabricator
Branch
draft3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18497
Build 24909: Run Core Tests
Build 24908: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Sep 18 2017, 11:52 PM
This revision was automatically updated to reflect the committed changes.