-
-
Notifications
You must be signed in to change notification settings - Fork 938
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
Make backup on flash an option #4360
Conversation
haslinghuis
commented
Feb 25, 2025
•
edited
Loading
edited
- adds option for backup during flashing (disabled, enabled, ask)
- enabled is default
- add error handling in filesystem API if there is no file handler available
- add error handling for backup process
- increase timeout to enter cli for backup command to reduce corruption of resulting config
- remove some unused messages
- fix parameter for i18n message
- refactor to make sonar happy
✅ Deploy Preview for origin-betaflight-app ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
4ce022e
to
c3120c2
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Sorry need to move default to main.js (as now it adds default in option). |
This comment was marked as outdated.
This comment was marked as outdated.
i don't know that i would keep this option ON. (but also like
|
Perhaps:
Not for now: |
|
220c09d
to
8220ca9
Compare
@nerdCopter think have addressed your review |
ccb7ce9
to
4e4f466
Compare
edit: i think |
This comment was marked as outdated.
This comment was marked as outdated.
@nerdCopter should overwrite once set. Saves for me (changing or refreshing tab in browser) |
This comment was marked as outdated.
This comment was marked as outdated.
Hmmm, works locally but not in preview 🤯 |
This comment was marked as outdated.
This comment was marked as outdated.
edit: in retrospect, i think |
Rebased on master |
e47edc5
to
52d8ed2
Compare
Testing on my end so far so good. I'll try and put it through the wringer tonight. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All options work as they should.
The only thing I am a little confused about is that when using Backup Enabled
, and the file picker dialog is closed, the Configurator won't open the selection for the DFU device (this seems appropriate), but the board stays in DFU mode. It'd be nice (if possible) to automatically exit DFU after that
@VitroidFPV , just in case: if you accidentally click out of chrome, the DFU dialog disappears. refresh seems to be the only answer. i re-tested your observation by resetting my permissions. refreshing, backup-enabled. canceled the save dialog. STM32 DFU popped up for me. Debain Linux, Chrome. |
|