Page MenuHomePhabricator

Remove the old, non-modular Excel export workflow from Maniphest
ClosedPublic

Authored by epriestley on Jan 29 2018, 4:59 PM.
Tags
None
Referenced Files
F14055833: D18961.diff
Sat, Nov 16, 4:13 PM
F14047512: D18961.id.diff
Thu, Nov 14, 4:06 AM
F14025951: D18961.id45461.diff
Thu, Nov 7, 9:31 PM
F14025034: D18961.id45461.diff
Thu, Nov 7, 2:09 PM
F14024960: D18961.id45461.diff
Thu, Nov 7, 1:42 PM
F13994388: D18961.id45461.diff
Wed, Oct 23, 5:49 AM
F13968472: D18961.diff
Oct 16 2024, 9:15 PM
Unknown Object (File)
Oct 13 2024, 2:47 AM
Subscribers
None

Details

Summary

Depends on D18960. Ref T13049. Now that Maniphest fully supports "Export Data", remove the old hard-coded version.

This is a backward compatibility break with the handful of installs that might have defined a custom export by subclassing ManiphestExcelFormat. I suspect this is almost zero installs, and that the additional data in the new format may serve most of the needs of this tiny number of installs. They can upgrade to ExportEngineExtensions fairly easily if this isn't true.

Test Plan
  • Viewed Maniphest, no longer saw the old export workflow.
  • Grepped for export and similar strings to try to hunt everything down.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable