Skip to content

Add default name and folder to export excel file#413

Merged
LBeghini merged 2 commits into
mainfrom
409-add-default-name-and-folder-to-export-excel-file
Oct 20, 2023
Merged

Add default name and folder to export excel file#413
LBeghini merged 2 commits into
mainfrom
409-add-default-name-and-folder-to-export-excel-file

Conversation

@fsoupimenta

Copy link
Copy Markdown
Member

Closes #409

@fsoupimenta fsoupimenta added the update It's working, but could be better label Oct 19, 2023
@fsoupimenta fsoupimenta self-assigned this Oct 19, 2023
@fsoupimenta

Copy link
Copy Markdown
Member Author

@GraphFilter/version-2-0

When analyzing the structure of the export, I realized that the get_name_from_save_dialog method was a bit out of place in the Controller class, so i took the liberty of throwing the method into the utils_file, leaving it there reusable for the export that is carried out in the toolbar (class exports_graphs_to).

What do you guys think?

@LBeghini LBeghini force-pushed the 409-add-default-name-and-folder-to-export-excel-file branch from 40383db to ac491db Compare October 20, 2023 01:54
@LBeghini LBeghini merged commit 2956294 into main Oct 20, 2023
@LBeghini LBeghini deleted the 409-add-default-name-and-folder-to-export-excel-file branch October 20, 2023 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

update It's working, but could be better

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add default name and folder to export Excel file

3 participants