@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Restore "Summary" and "Test Plan" to initial mail for non-draft configurations
Summary: See PHI210. Ref T2543. Currently, we don't set this flag if you have prototypes off and don't get any of the new draft stuff, so the mail drops some of the details it is supposed to have.
Test Plan: Disabled prototypes, created a revision, saw summary / test plan in the initial mail.
Reviewers: amckinley
Reviewed By: amckinley
Maniphest Tasks: T2543
Differential Revision: https://secure.phabricator.com/D18771
+7
+7
src/applications/differential/editor/DifferentialTransactionEditor.php
+7
src/applications/differential/editor/DifferentialTransactionEditor.php
···
1612
1612
1613
1613
$xactions[] = $xaction;
1614
1614
}
1615
+
} else {
1616
+
// If this revision is being created into some state other than "Draft",
1617
+
// this is the first broadcast and should include sections like "SUMMARY"
1618
+
// and "TEST PLAN".
1619
+
if ($this->getIsNewObject()) {
1620
+
$this->firstBroadcast = true;
1621
+
}
1615
1622
}
1616
1623
1617
1624
return $xactions;