Page MenuHomePhabricator

Allow revisions to be held as drafts, even after builds finish
ClosedPublic

Authored by epriestley on Oct 27 2017, 4:10 PM.
Tags
None
Referenced Files
F12834659: D18737.id44985.diff
Thu, Mar 28, 2:44 PM
Unknown Object (File)
Feb 13 2024, 7:45 AM
Unknown Object (File)
Feb 12 2024, 7:39 PM
Unknown Object (File)
Feb 12 2024, 7:53 AM
Unknown Object (File)
Feb 1 2024, 11:25 PM
Unknown Object (File)
Jan 25 2024, 12:41 AM
Unknown Object (File)
Jan 23 2024, 2:02 AM
Unknown Object (File)
Jan 8 2024, 5:43 PM
Subscribers
None

Details

Summary

Ref T2543. Instead of autosubmitting revisions to "Needs Review" when builds finish, allow them to be held in "Draft" indefinitely.

There's currently no UI for this. I plan to just expose it as arc diff --draft for now, in a followup change.

Test Plan
  • Created a revision (via Conduit) with "hold as draft", saw it hold as draft after builds finished.
  • Created a revision (normally), saw it autosubmit after builds finished.
  • Requested review of a "hold as draft" revision to kick it out of draft state.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/differential/storage/DifferentialRevision.php
716

Unused argument.

Should be return $this->setProperty(self::PROPERTY_DRAFT_HOLD, $hold);?

This revision now requires changes to proceed.Oct 28 2017, 3:27 PM
  • Fix setHoldAsDraft() implementation to respect parameter.
This revision is now accepted and ready to land.Oct 31 2017, 2:48 PM
This revision was automatically updated to reflect the committed changes.