Closed NotherNgineer closed 7 months ago
Sounds good. Do you prefer to do some separated PR's for the on packet beep support?
Sounds good. Do you prefer to do some separated PR's for the on packet beep support?
If it sounds good, I'll go ahead and add beep support to some apps in this PR. I've changed it to a Draft until then. Thanks.
I've updated this PR to include ERT and Radiosonde support for this new "Beep on RX packet" setting, and marked the PR "ready to review". To avoid making this PR any larger, I propose to implement beep-on-packet support in additional apps in future follow-on PR's.
Note: If Beep is desired in some apps but not in others, set "Beep on RX Packets" to True, and turn down the volume to 0 in apps where the beep is not wanted.
(Support for beep-on-RX-packet in other apps to follow in separate PR's)
This PR should not affect beep settings in the the Level and Looking Glass apps (technically these apps are not even looking for "packets").
Test firmware on Discord: https://discord.com/channels/719669764804444213/722101917135798312/1223324351667896450