Homec4science

Phriction - move "move" to modern editor + transactions

Authored by Bob Trahan <btrahan@phacility.com> on Nov 5 2014, 21:48.

Description

Phriction - move "move" to modern editor + transactions

Summary:
Ref T4029. Much like D10756, D10761 this does the bare minimum to get things in there. I have a sticky with "TODOs" about moving the error-checking business logic into the editor in all three cases.

Up next - policy...

Test Plan: moved a document and it worked! verified no feed story. verified both documents involved looked good

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T4029

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

Details

Committed
Bob Trahan <btrahan@phacility.com>Nov 5 2014, 21:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbdbb77183074: Phriction - move delete to modern editor + transactions
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPHdd7d8cf910e6: Phriction - move "move" to modern editor + transactions (authored by Bob Trahan <btrahan@phacility.com>).Nov 5 2014, 21:48