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

CPN: user configurable audio mute in radio settings. Supports PR #3703 #3710

Merged
merged 2 commits into from
Jun 26, 2023

Conversation

mha1
Copy link
Contributor

@mha1 mha1 commented Jun 22, 2023

Adds user configurable audio mute in CPN's radio setting. Supports PR #3703

@pfeerick pfeerick added the companion Related to the companion software label Jun 22, 2023
@mha1
Copy link
Contributor Author

mha1 commented Jun 23, 2023

Updated to make checkbox only visible for radios with AUDIO_MUTE_GPIO pin (see target/hal.h files):

  • All color lcd radios (including NV14) except Horus X12S
  • TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8

@pfeerick
Copy link
Member

@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.

@elecpower
Copy link
Collaborator

@pfeerick looks okay

Copy link
Member

@pfeerick pfeerick left a 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

@pfeerick pfeerick changed the base branch from main to 2.9 June 26, 2023 09:00
mha1 added 2 commits June 26, 2023 19:03
 - All color lcd radios (including NV14) except Horus X12S
 - TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8
@pfeerick pfeerick force-pushed the PR_CPN_audioMuteEnable branch from 3bbf291 to 2b5d245 Compare June 26, 2023 09:03
@pfeerick pfeerick added this to the 2.9 milestone Jun 26, 2023
@pfeerick pfeerick merged commit 8922052 into EdgeTX:2.9 Jun 26, 2023
pfeerick pushed a commit that referenced this pull request Jun 30, 2023
* 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
pfeerick added a commit that referenced this pull request Jul 1, 2023
* 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]>
@mha1 mha1 deleted the PR_CPN_audioMuteEnable branch July 6, 2023 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
companion Related to the companion software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants