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.

Sign up for GitHub

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

Jump to bottom

fix(cpn): switches default types #4641

Merged
merged 2 commits into from
Feb 26, 2024
Merged

fix(cpn): switches default types #4641

merged 2 commits into from
Feb 26, 2024

Conversation

elecpower
Copy link
Collaborator

Fixes #4640

Summary of changes:

  • load switch default for new settings file and not max positions
  • load defaults before reading yaml file to handle cases where yaml file does not include the switch settings

@elecpower elecpower added companion Related to the companion software bug/regression ↩️ A new version of EdgeTX broke something labels Feb 15, 2024
@elecpower elecpower added this to the 2.10 milestone Feb 15, 2024
@pfeerick pfeerick merged commit 7183490 into main Feb 26, 2024
3 checks passed
@pfeerick pfeerick deleted the elecpower/fix-4640-switches branch February 26, 2024 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug/regression ↩️ A new version of EdgeTX broke something companion Related to the companion software
Projects
None yet
Milestone
2.10
Development

Successfully merging this pull request may close these issues.

Incorrect switch settings in Companion 2.10.
2 participants
@elecpower @pfeerick