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

Autosave #32

Open
keysym opened this issue Apr 11, 2023 · 1 comment
Open

Autosave #32

keysym opened this issue Apr 11, 2023 · 1 comment
Labels
enhancement New feature or request feature
Milestone

Comments

@keysym
Copy link
Contributor

keysym commented Apr 11, 2023

As tarsila panics on errors, an autosave could be great to avoid losing progress.

Should autosave...

  • Overwrite the existing .tarsila OR write to a .tmp/.backup file to recover later in case of a panic?
  • Save on a time interval OR based on user events?
@yds12 yds12 added enhancement New feature or request feature labels Apr 12, 2023
@yds12
Copy link
Owner

yds12 commented Apr 12, 2023

Time interval I think would be better, if we decide to save after every edit event it will be too much writing on disc. We can have a const with the interval in milliseconds, so later it can be configurable via config file. I'd prefer it to save to a temp file, it's less intrusive for the user.

We probably need a crate to help finding a cross-platform way of getting a temp directory.

Also creating this ticket to remember to improve our error handling: #33

Let me know if you interested in working on this, so I can mark the ticket as assigned :)

@yds12 yds12 added this to the 0.2.0 milestone Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature
Projects
None yet
Development

No branches or pull requests

2 participants