Page MenuHomePhabricator

Phriction - move delete business logic from controlller to editor
ClosedPublic

Authored by btrahan on Nov 7 2014, 5:25 PM.
Tags
None
Referenced Files
F12841023: D10809.id.diff
Thu, Mar 28, 8:53 PM
F12839455: D10809.id25935.diff
Thu, Mar 28, 7:30 PM
F12838264: D10809.id25939.diff
Thu, Mar 28, 6:02 PM
F12838162: D10809.id25935.diff
Thu, Mar 28, 5:56 PM
Unknown Object (File)
Mon, Mar 25, 6:36 PM
Unknown Object (File)
Wed, Mar 20, 9:03 AM
Unknown Object (File)
Mon, Mar 4, 5:05 PM
Unknown Object (File)
Feb 8 2024, 5:46 AM
Subscribers

Details

Summary

Ref T4029. More cleanup and code consolidation for the long terms benefits.

Test Plan

found a document and opened up two browser tabs. Loaded delete dialog on both. Completed delete in one tab and noted document was properly deleted. Tried to complete delete in tab 2 and got an error message.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Phriction - move delete business logic from controlller to 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, 5:45 PM
This revision was automatically updated to reflect the committed changes.