-
-
Notifications
You must be signed in to change notification settings - Fork 895
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: expose OSD for virtual FC to pick up #3025
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Do you want to test this code? Here you have an automated build: |
window.TABS.osd = osd; | ||
window.OSD = OSD; |
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.
Maybe is better to modify in the VirtualFC file:
betaflight-configurator/src/js/VirtualFC.js
Line 179 in db99a98
const virtualOSD = OSD; |
Modify const virtualOSD = OSD
by const virtualOSD = TABS.osd
? In this way we don't duplicate here the variable. I did a little test yesterday but it not worked, but maybe I did something wrong.
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.
Good shout. Will grab a look
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.
Can we use export / import?
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.
Eventually we can, but would need move virtual fc to modules first. But at the moment I'm trying to do similar thing to GUI first.
AUTOMERGE: (FAIL)
|
Closes #3023