Skip to content

Fix Digirig Lite tone PTT greyed out on Android#380

Open
chrissnell wants to merge 1 commit into
mainfrom
feature/fix-digirig-tone-ptt-android-device-step
Open

Fix Digirig Lite tone PTT greyed out on Android#380
chrissnell wants to merge 1 commit into
mainfrom
feature/fix-digirig-tone-ptt-android-device-step

Conversation

@chrissnell

Copy link
Copy Markdown
Owner

Problem

On a fresh install, the new Digirig Lite tone PTT method can't be selected on Android — the operator gets stuck on a device-picker step with no devices and a permanently greyed-out Save button.

Cause

Digirig Lite tone PTT keys the radio with an audio tone and needs no serial/HID device (the same as VOX, and the same as desktop's digirig_tone). But needsDevice() in Ptt.svelte only skipped the device-selection step for Android VOX (ptt_method 4), not tone PTT (ptt_method 5). Selecting it routed into DialogChangeDevice, whose Android device source returns an empty list for any method with no deviceClass — so Save (gated on a selected device) stayed disabled and the method could never be applied.

The Android methods table already documents that tone PTT carries no deviceClass; only the needsDevice gate was out of sync.

Fix

Treat Android ptt_method 5 like VOX: skip the device step and persist directly. One-line change mirroring the existing desktop digirig_tone handling.

All 365 web unit tests pass.

The Android "Digirig Lite (tone PTT)" method keys the radio via an audio
tone and needs no serial/HID device, but needsDevice() only skipped the
device-selection step for VOX (ptt_method 4), not tone PTT (ptt_method 5).
Selecting it pushed the operator into the device picker, which lists no
devices for a method with no deviceClass, leaving Save permanently
disabled so the method could never be applied.

Treat ptt_method 5 like VOX and like desktop's digirig_tone: skip the
device step and persist directly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant