Homec4science

Phriction - move delete to modern editor + transactions

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

Description

Phriction - move delete to modern editor + transactions

Summary:
Ref T4029. Much like D10756 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 both cases.

Up next - move actions...

Test Plan: deleted a document and it worked! verified proper feed story.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: shadowhand, Korvin, epriestley

Maniphest Tasks: T4029

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

Details

Committed
Bob Trahan <btrahan@phacility.com>Nov 5 2014, 21:37
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf7fbf9ccc174: Change usernames to `sort64` to make them case-insensitive
Branches
Unknown
Tags
Unknown

Event Timeline

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