Page MenuHomePhabricator

Phriction - move "move" business logic to the editor
ClosedPublic

Authored by btrahan on Nov 7 2014, 7:14 PM.
Tags
None
Referenced Files
F18831364: D10812.diff
Sat, Oct 25, 12:34 PM
F18822868: D10812.id25946.diff
Thu, Oct 23, 8:59 AM
F18821696: D10812.diff
Thu, Oct 23, 1:18 AM
F18817421: D10812.id25944.diff
Tue, Oct 21, 3:24 PM
F18811829: D10812.id.diff
Mon, Oct 20, 5:12 AM
F18809350: D10812.diff
Sun, Oct 19, 1:13 PM
F18745632: D10812.id25946.diff
Oct 3 2025, 8:29 AM
F18671136: D10812.id25944.diff
Sep 25 2025, 2:48 AM
Subscribers

Details

Summary

Ref T4029. Even more code consolidation and cleanup for the long term benefits!

Test Plan

moved a page successfully. tried to move a page to an existing page and got an error. did the two tab trick to try to move a deleted page and got an error.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Phriction - move "move" business logic to the editor.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: epriestley, chad.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 7 2014, 7:42 PM
This revision was automatically updated to reflect the committed changes.