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
F20233894: D21401.diff
Mon, Jun 8, 2:15 AM
Unknown Object (File)
Mar 16 2026, 10:57 PM
Unknown Object (File)
Mar 15 2026, 10:54 AM
Unknown Object (File)
Mar 8 2026, 11:18 PM
Unknown Object (File)
Dec 4 2025, 7:54 PM
Unknown Object (File)
Dec 1 2025, 3:08 AM
Unknown Object (File)
Nov 23 2025, 4:56 AM
Unknown Object (File)
Nov 22 2025, 4:06 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.