Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More defensive approach when it comes to saving files #6

Closed
wouterfennis opened this issue Feb 27, 2021 · 0 comments · Fixed by #23
Closed

More defensive approach when it comes to saving files #6

wouterfennis opened this issue Feb 27, 2021 · 0 comments · Fixed by #23
Assignees
Labels
bug Something isn't working
Milestone

Comments

@wouterfennis
Copy link
Owner

When you currently save a spreadsheets that already exists on disk the program will fail.
We need more defensive programming to prevent this.
The https://github.com/wouterfennis/SpreadsheetWriter needs to be more robust and give a clear message if the saving went successful or that an error did occur.
Cifra will have to handle the situation where a file can't be saved.
Maybe ask for an other name or try again.

@wouterfennis wouterfennis added the bug Something isn't working label Feb 27, 2021
@wouterfennis wouterfennis added this to the MVP milestone Feb 27, 2021
@wouterfennis wouterfennis self-assigned this Feb 27, 2021
@wouterfennis wouterfennis added this to In progress in Cifra Feb 27, 2021
@wouterfennis wouterfennis linked a pull request May 14, 2021 that will close this issue
@wouterfennis wouterfennis removed this from In progress in Cifra Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant