Page MenuHomePhabricator

Phriction - move "move" to modern editor + transactions
ClosedPublic

Authored by btrahan on Oct 30 2014, 8:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 12:53 AM
Unknown Object (File)
Sat, Mar 23, 2:57 AM
Unknown Object (File)
Thu, Mar 14, 6:05 PM
Unknown Object (File)
Fri, Mar 8, 1:54 PM
Unknown Object (File)
Fri, Mar 8, 9:53 AM
Unknown Object (File)
Feb 9 2024, 1:58 PM
Unknown Object (File)
Feb 3 2024, 1:13 AM
Unknown Object (File)
Jan 25 2024, 5:16 AM
Subscribers

Details

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

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Phriction - move "move" to modern editor + transactions.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
btrahan added subscribers: Korvin, epriestley.

lets see what we get here...

and let's see what this does

Sorry about the git hyjinx. This should be good to review now.

epriestley edited edge metadata.

Seems like a reasonable starting point.

This revision is now accepted and ready to land.Oct 31 2014, 11:36 PM
This revision was automatically updated to reflect the committed changes.