Page MenuHomePhabricator

Provide a more general "Author" transaction for Differential
ClosedPublic

Authored by epriestley on Mar 3 2021, 11:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 28 2024, 11:15 PM
Unknown Object (File)
Jan 10 2024, 1:12 PM
Unknown Object (File)
Jan 6 2024, 5:26 PM
Unknown Object (File)
Dec 26 2023, 10:51 AM
Unknown Object (File)
Dec 19 2023, 5:34 PM
Unknown Object (File)
Dec 7 2023, 5:39 PM
Unknown Object (File)
Nov 30 2023, 1:45 AM
Unknown Object (File)
Nov 26 2023, 12:19 AM
Subscribers

Details

Summary

Ref T13628. Currently, Differential has a "Commandeer" action, but no way to edit the author otherwise.

This is largely archaic: there is no reason to prevent editing the author, and this makes it difficult to undo mistakes if you commandeer by accident.

Instead, provide a normal "Author" field and a "Foist Upon" action, similar to the "Owner" and "Claim/Assign" fields in Maniphest.

Test Plan
  • Applied author transactions as the old author ("foisted"), the new author ("commandeered"), and an arbitrary third party ("changed author").
  • Tried to unassign author, etc.
  • Viewed stories in feed and transaction timeline.
  • Saw sensible automatic reviewer changes.
  • Used existing "Commandeer" action, which is unchanged.
  • Called "transaction.search" and saw reasonable transaction values.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 3 2021, 11:48 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.
epriestley edited reviewers, added: bjshively; removed: epriestley.