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

chore: remove support for binary storage format #2900

Merged
merged 1 commit into from
Dec 23, 2022
Merged

Conversation

raphaelcoeffic
Copy link
Member

This PR removes in-radio support for binary storage format and related conversions. Following firmwares will be YAML only.

@froqstar
Copy link

So to migrate from OpenTX one first needs to flash EdgeTX<=2.8, and then the latest release, correct?

@raphaelcoeffic
Copy link
Member Author

So to migrate from OpenTX one first needs to flash EdgeTX<=2.8, and then the latest release, correct?

Yes, or you use Companion, which will retain compatibility to the binary formats and continue to support them.

@pfeerick
Copy link
Member

pfeerick commented Dec 19, 2022

Now is probably a good time to flip the EdgeTX version over to 2.9 as well? Not that I should have done it earlier... 😆 And did you notice the PR number? 🤓

@gagarinlg
Copy link
Member

gagarinlg commented Dec 19, 2022

Now is probably a good time to flip the EdgeTX version over to 2.9 as well? Not that I should have done it earlier... 😆 And did you notice the PR number? 🤓

We can do it Microsoft style, we keep the version number for years and add a date code to it

@pfeerick
Copy link
Member

Didn't you hear... they decided three years after positively stating that "Windows 11 will be the last version ever" that "Windows 12 is now being worked on" 🤦 ... but yeah, date code is certainly another way to do it...

@raphaelcoeffic raphaelcoeffic added this to the 2.9 milestone Dec 19, 2022
@raphaelcoeffic
Copy link
Member Author

Now is probably a good time to flip the EdgeTX version over to 2.9 as well?

Ideally, this should be the first commit after forking a stable branch 🤓

@pfeerick pfeerick merged commit f9bac7b into main Dec 23, 2022
@pfeerick pfeerick deleted the remove-binary-conv branch December 23, 2022 03:06
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.

4 participants