Page MenuHomePhabricator

Remove obsolete, nonfunctional draft auto-saving in Phriction
ClosedPublic

Authored by epriestley on Sep 11 2018, 4:39 PM.
Tags
None
Referenced Files
F13061757: D19661.id.diff
Fri, Apr 19, 7:56 PM
F13061750: D19661.diff
Fri, Apr 19, 7:55 PM
Unknown Object (File)
Thu, Apr 11, 7:54 PM
Unknown Object (File)
Thu, Apr 11, 9:52 AM
Unknown Object (File)
Tue, Apr 2, 11:18 PM
Unknown Object (File)
Tue, Apr 2, 9:09 PM
Unknown Object (File)
Sat, Mar 30, 6:35 PM
Unknown Object (File)
Feb 24 2024, 5:04 PM
Subscribers
None

Details

Summary

Depends on D19660. Ref T5811. Ref T13077.

Long ago, if you started editing a Phriction document but didn't save it, we'd save the draft in the background as part of the preview.

D11169 updated the preview to use shared infrastructure and broke this function, since we never save drafts.

Since this doesn't work right now, I want to add another thing called "draft", and the future of this feature should be more integrated with modern drafts and EditEngine (which fixed some bugs related to versioning), just get rid of this code for the moment.

Test Plan

Edited documents. This code doesn't do anything since D11169, so no behavior changed.

Diff Detail

Repository
rP Phabricator
Branch
wiki3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20791
Build 28276: Run Core Tests
Build 28275: arc lint + arc unit