Homec4science

Begin adding "pro" modular transaction fields to Differential

Authored by epriestley <git@epriestley.com> on Dec 13 2016, 22:03.

Description

Begin adding "pro" modular transaction fields to Differential

Summary:
Ref T11114. Currently, all of Differential is extremely custom CustomFields. I want to back away from that somewhat and leverage more EditEngine / ModularTransactions infrastructure.

This allows EditEngine, ModularTransactions, and CustomFields to coexist in an uneasy peace. The "EditPro" controller applies a different edit than the CustomFields do, but everything works out in the end. I think.

Hopefully the horrible mess I am creating here will be short-lived.

Test Plan:

  • Edited a revision with the normal editor.
  • Edited a revision with the pro editor.
  • Created a revision with arc diff.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11114

Differential Revision: https://secure.phabricator.com/D17044

Details

Committed
epriestley <git@epriestley.com>Dec 13 2016, 23:50
Pushed
aubortJan 31 2017, 17:16
Parents
rPHeda64b85497b: Add a very basic EditPro controller for Differential
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH0906bf547b9a: Begin adding "pro" modular transaction fields to Differential (authored by epriestley <git@epriestley.com>).Dec 13 2016, 23:50