search

allenai / AMPT

Aquatic Mammal Photogrammetry Tool
Apache License 2.0
3 stars 0 forks source link

Add an "Export Configuration" option to the hamburger menu. #46

Closed APatrickJ closed 3 years ago

APatrickJ commented 3 years ago

This feature that allows the user to export their current configuration to a ZIP file to the location of their choosing. It includes the JSON and CSV configuration files in the resulting ZIP file.

Fixes #16

APatrickJ commented 3 years ago

Please put files into a directory on export.

Fixed in b9a3cdf

palbee commented 3 years ago

I'm not wild about the ConfigurationExportLauncher name. The name and documentation suggest that the class just coordinates kicking off the workflow. Instead it is the embodyment of the workflow.

APatrickJ commented 3 years ago

I'm not wild about the ConfigurationExportLauncher name. The name and documentation suggest that the class just coordinates kicking off the workflow. Instead it is the embodyment of the workflow.

Naming is hard :-/

Renamed to ConfigurationExportWorkflow in fd5464d. Happy to take other suggestions into consideration.

Will rename related files (ConfigurationImportLauncher, ConfigurationRestoreLauncher) in another PR.