Differential D18957

Make the data export format selector remember your last setting
ClosedPublic
Actions

Authored by epriestley on Jan 29 2018, 3:35 PM.

Details

Reviewers

amckinley

Maniphest Tasks

T13049: Plans: Data Exporters (Excel, JSON, CSV, etc.)

Commits

rP61b8c12970be: Make the data export format selector remember your last setting

Summary

Depends on D18956. Ref T13049. Make the "Export Format" selector sticky.

This is partly selfish, since it makes testing format changes a bit easier.

It also seems like it's probably a good behavior in general: if you export to Excel once, that's probably what you're going to pick next time.

Test Plan

Exported to excel. Exported again, got excel as the default option.

Diff Detail

Repository

rP Phabricator

Branch

export7

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 19240
Build 25996: Run Core Tests
Build 25995: arc lint + arc unit

Event Timeline

epriestley created this revision.Jan 29 2018, 3:35 PM

Harbormaster completed remote builds in B19240: Diff 45456.Jan 29 2018, 3:36 PM

epriestley requested review of this revision.Jan 29 2018, 3:36 PM

epriestley added a child revision: D18958: When PHPExcel is not installed, detect it and provide install instructions.Jan 29 2018, 3:57 PM

Cool that this is so easy!

This revision is now accepted and ready to land.Jan 29 2018, 9:32 PM

This could probably be wrapped in something to make it a little slimmer if we start doing more of it, but I think we only have ~2-3 places where we do this right now (the other one that springs to mind is that we save projects you've recently used to set policies in the policy control).

Closed by commit rP61b8c12970be: Make the data export format selector remember your last setting (authored by epriestley). · Explain WhyJan 30 2018, 12:02 AM

This revision was automatically updated to reflect the committed changes.