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
F15400737: D18961.id.diff
Mon, Mar 17, 1:40 PM
F15393907: D18961.diff
Sat, Mar 15, 11:15 PM
F15390254: D18961.id.diff
Sat, Mar 15, 6:04 AM
F15384330: D18961.diff
Fri, Mar 14, 7:32 PM
F15367360: D18961.id45461.diff
Tue, Mar 11, 5:53 PM
F15285633: D18961.id45485.diff
Tue, Mar 4, 2:56 PM
Unknown Object (File)
Mon, Feb 24, 11:27 PM
Unknown Object (File)
Feb 18 2025, 9:34 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