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

fix(cpn): Edit radio settings application crash #3978

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

elecpower
Copy link
Collaborator

@elecpower elecpower commented Aug 28, 2023

Fixes #3977

Bug in global functions when building lists containing timer related entries introduced by #3956

Summary of changes:

  • test for existence of a model pointer in function parameters before calling model functions

@elecpower elecpower added companion Related to the companion software bug/regression ↩️ A new version of EdgeTX broke something labels Aug 28, 2023
@elecpower elecpower added this to the 2.10 milestone Aug 29, 2023
@pfeerick pfeerick changed the title fix(cpn): edit radio settings application crash fix(cpn): Edit radio settings application crash Aug 29, 2023
@pfeerick
Copy link
Member

Ouch! At least we know there's one bleeding edge user! 😁

@pfeerick pfeerick merged commit ceba626 into main Aug 29, 2023
3 checks passed
@pfeerick pfeerick deleted the elecpower/cpn-fix-3977-crash branch August 29, 2023 08:42
@elecpower
Copy link
Collaborator Author

Yeah my buggy code is at least fresh in my mind 🤪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/regression ↩️ A new version of EdgeTX broke something companion Related to the companion software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Companion Edit Radio Settings crashes application
2 participants