Page MenuHomePhabricator

When you "Request Review" of a draft revision, change the button text from "Submit Quietly" to "Publish Revision"
ClosedPublic

Authored by epriestley on Nov 15 2018, 1:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 27, 6:29 AM
Unknown Object (File)
Oct 20 2024, 10:40 PM
Unknown Object (File)
Oct 20 2024, 4:00 AM
Unknown Object (File)
Sep 9 2024, 3:18 PM
Unknown Object (File)
Sep 6 2024, 4:21 AM
Unknown Object (File)
Aug 28 2024, 6:35 PM
Unknown Object (File)
Aug 22 2024, 7:55 PM
Unknown Object (File)
Aug 22 2024, 5:46 PM
Subscribers
None

Details

Summary

See PHI975. Ref T13216. Ref T2543. Previously, see D19204 and PHI433.

When you're acting on a draft revision, we change the button text to "Submit Quietly" as a hint that your actions don't generate notifications yet.

However, this isn't accurate when one of your actions is "Request Review", which causes the revision to publish.

Allow actions to override the submit button text, and make the "Request Review" action change the button text to "Publish Revision".

The alternative change I considered was to remove the word "Quietly" in all cases.

I'm not thrilled about how complex this change is to adjust one word, but the various pieces are all fairly clean individually. I'm not sure we'll ever be able to use it for anything else, but I do suspect that the word "Quietly" was the change in D19204 with the largest effect by far (see T10000).

Test Plan
  • Created a draft revision. Saw "Submit Quietly" text.
  • Added a "Request Review" action, saw it change to "Publish Revision".
  • Reloaded page, saw stack saved and "Publish Revision".
  • Removed action, saw "Submit Quietly".
  • Repeated on a non-draft revision, button stayed put as "Submit".
  • Submitted the various actions, saw them have the desired effects.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/owners/query/PhabricatorOwnersPackageQuery.php
231

Unrelated qsprintf() fix, repro is to load revisions with some (particular?) packages configured.

This revision is now accepted and ready to land.Nov 16 2018, 12:07 AM
This revision was automatically updated to reflect the committed changes.