-
-
Notifications
You must be signed in to change notification settings - Fork 898
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
New crash flag in 4.1 jostles the disarm flags #1426
Conversation
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.
Thanks @Docteh, I had forgotten to add this to configurator.
locales/en/messages.json
Outdated
@@ -789,6 +789,10 @@ | |||
"message": "Runway Takeoff Prevention has been triggered", | |||
"description": "Description of the RUNAWAY_TAKEOFF arming disable flag" | |||
}, | |||
"initialSetupArmingDisableFlagsTooltipCRASH": { | |||
"message": "Ouch!", |
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.
It's funny, but I don't know if this is the correct description :)
I usually took them from here: https://github.com/betaflight/betaflight/wiki/Arming-Sequence-&-Safety but it seems it has neither been added to the wiki.
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.
"Disarmed via crash detection"
"Apparently, we've crashed"
"Crash has been detected"
Its a fairly recent status flag, that is very unlikely to show on the configurator.
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.
If we set it to something funny now I am almost certain we will forget to change it back before the next release. "Disarmed via crash detection" sounds good.
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.
thanks for choosing. :)
related to betaflight/betaflight@760a524
Note: initialSetupArmingDisableFlagsTooltipCRASH.message might need to get handled somewhere. Maybe people could just not crash with the configurator open...