Page MenuHomePhabricator

Phriction - move delete business logic from controlller to editor
ClosedPublic

Authored by btrahan on Nov 7 2014, 5:25 PM.

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.