Page MenuHomePhabricator

Expand Revision transaction API to allow actions to vary more broadly based on the viewer and revision state
ClosedPublic

Authored by epriestley on Jul 9 2020, 7:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 6:48 PM
Unknown Object (File)
Oct 16 2024, 6:54 AM
Unknown Object (File)
Sep 27 2024, 8:22 PM
Unknown Object (File)
Sep 27 2024, 7:10 PM
Unknown Object (File)
Sep 27 2024, 6:45 PM
Unknown Object (File)
Sep 3 2024, 9:07 AM
Unknown Object (File)
Aug 29 2024, 10:00 AM
Unknown Object (File)
Aug 29 2024, 12:03 AM
Subscribers
None

Details

Summary

See PHI1810. Build toward support for "Request Review" by non-authors on drafts, to forcefully pull a revision out of draft.

Currently, some action strings can't vary based on revision state or the current viewer, so this "pull out of draft" action would have to either: say "Request Review"; or be a totally separate action.

Neither seem great, so allow the labels and messages to vary based on the viewer and revision state.

Test Plan

Grepped for affected symbols, see followup changes.

Diff Detail

Repository
rP Phabricator
Branch
request1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24760
Build 34147: Run Core Tests
Build 34146: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jul 9 2020, 9:20 PM
This revision was automatically updated to reflect the committed changes.