-
-
Notifications
You must be signed in to change notification settings - Fork 356
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
CPN: user configurable audio mute in radio settings. Supports PR #3703 #3710
Conversation
Updated to make checkbox only visible for radios with AUDIO_MUTE_GPIO pin (see target/hal.h files):
|
@elecpower Can you give this a glance over, and let me know if you're happy with it? I'll give the combined two PRs a try tomorrow. |
@pfeerick looks okay |
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.
LGTM and has Neil's blessing also ;) Will merge after #3703
- All color lcd radios (including NV14) except Horus X12S - TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8
3bbf291
to
2b5d245
Compare
* CPN: user configurable audio mute in radio settings. Supports PR #3703 * make checkbox only visible for radios with AUDIO_MUTE_GPIO pin: - All color lcd radios (including NV14) except Horus X12S - TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8
* feat(cpn): User configurable audio mute in radio settings (#3710) * CPN: user configurable audio mute in radio settings. Supports PR #3703 * make checkbox only visible for radios with AUDIO_MUTE_GPIO pin: - All color lcd radios (including NV14) except Horus X12S - TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8 * chore: Add Companion translations --------- Co-authored-by: Michael <[email protected]>
Adds user configurable audio mute in CPN's radio setting. Supports PR #3703