betaflight-configurator
betaflight-configurator copied to clipboard
betaflight
Enable or disable features according to port configuration
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Debug-Android Betaflight-Configurator-Debug-Linux Betaflight-Configurator-Debug-macOS Betaflight-Configurator-Debug-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Debug-Android Betaflight-Configurator-Debug-Linux Betaflight-Configurator-Debug-macOS Betaflight-Configurator-Debug-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Debug-Android Betaflight-Configurator-Debug-Linux Betaflight-Configurator-Debug-macOS Betaflight-Configurator-Debug-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
The idea is great, but has some sense to define the port and disable the feature?
If not, to me, the correct way is to remove completely the "feature" in the firmware, and make it enable it simply when defined the port.
AUTOMERGE: (FAIL)
- github identifies PR as mergeable -> PASS
- assigned to a milestone -> PASS
- cooling off period lapsed -> PASS
- commit count less or equal to three -> PASS
-
Don't mergelabel NOT found -> PASS - at least one
RN:label found -> PASS -
Testedlabel found -> FAIL - assigned to an approver -> PASS
- approver count at least three -> FAIL
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
Kudos, SonarCloud Quality Gate passed! 
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
![sonarqubecloud[bot] avatar](https://avatars.githubusercontent.com/in/12526?v=4 "Published by a pub.dev verified publisher"){kind=small}
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!
As requested, I can approve it if you want, but as I said, I think this is not the correct approach.
This is a workaround more that we need to maintain in the Configurator.
I think that if we want that something is enabled or disabled automatically, is better to remove the feature in the firmware and simply make it enabled by default. Or if we want to maintain the feature, enable it in the firmware not in the Configurator when it receives the MSP command.
The problem with the Configurator is that we need to maintain backward compatibility, if the things change between versions, we need to maintain old and new code. If we do this in the firmware part and it changes in a future, the old code is simply removed and new code replaces it.
i would agree, sounds reasonable
Could VTX tab be hided with the same logic in this PR? Discussed it with @limonspb a week ago and was going to open own PR.
P.S. I like @McGiverGim proposion
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
Do you want to test this code? Here you have an automated build: Betaflight-Configurator-Android Betaflight-Configurator-Linux Betaflight-Configurator-macOS Betaflight-Configurator-Windows WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!