Skip to content

Backup function #900

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

marsman7
Copy link
Contributor

Data backup and restore

A button has been added to the main menu.

A data backup to a ZIP archive can be created in the submenu. The ‘config.json’ file is also created. The ZIP archive is not cached in the file system but downloaded directly.

A ZIP archive is uploaded for data recovery. This is not stored temporarily in the file system but unpacked directly. Files that already exist are overwritten.

Only uncompressed ZIP archives are processed (store mode).

Translated with DeepL.com (free version)

Copy link
Collaborator

@fvanroie fvanroie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Translations are not needed, they are merged in using en.json on https://crowdin.com/project/openhasp/

Copy link
Collaborator

@fvanroie fvanroie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to have an option like HASP_USE_BACKUP to enable/disable the new code.
Now this overhauls a chunk of code that needs to be validated first.

I'd like to keep the old code with a #IF defined() untill validated.

@marsman7
Copy link
Contributor Author

I'll see what I can do.

@marsman7
Copy link
Contributor Author

I have found an problem. I am still looking for the error. It is related to the call of the function “timeSetConfig(settings[FPSTR(FP_TIME)]);” in the file “hasp_config.cpp”.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants