Add common aircraft frequencies to airport channel import - #141
Open
emuehlstein wants to merge 2 commits into
Open
Add common aircraft frequencies to airport channel import#141emuehlstein wants to merge 2 commits into
emuehlstein wants to merge 2 commits into
Conversation
Adds an optional set of common/itinerant aircraft frequencies (Guard, UNICOM, air-to-air, Multicom, SAR, etc.) to the Channel Wizard airport import flow. - Service: generateAirportChannels now accepts a selected subset of common frequencies. In single-zone mode they are appended to the Airports zone; in individual-zone mode they are placed in a separate Aircraft zone. Channels are receive-only (forbidTx, Analog, 25kHz). - UI: the 'Add common aircraft frequencies' checkbox is available in both zone-grouping modes, with a select/deselect list (all selected by default) plus Select All / Deselect All. - Tests: updated and added coverage for single-zone append, separate Aircraft zone in individual mode, partial subset selection, empty list, and receive-only channels.
Owner
|
It will make a build and deploy it to test environment... If you are ready and it looks good there let me know we can then merge it. |
emuehlstein
had a problem deploying
to
github-pages
July 28, 2026 15:15 — with
GitHub Actions
Failure
Owner
|
/preview |
|
Mirrored for preview/deploy testing: #145 |
Owner
|
https://neonplug.app/test/preview-pr-141/ <- so this way works.. I have to issue preview but that tells it to make a copy under me and deploy it for testing. |
|
Mirrored for CI/deploy testing: #145 Preview will be live at https://neonplug.app/test/preview-pr-141/ once the build finishes (usually a few minutes). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds an optional set of common / itinerant aircraft frequencies (Guard, UNICOM, air-to-air, Multicom, SAR, etc.) to the Channel Wizard airport import flow. These are the itinerant frequencies listed on the RadioReference Aircraft wiki, useful to have alongside nearby airport channels.
What's included
generateAirportChannels) — now accepts a selected subset of common frequencies instead of a boolean:Airportszone.Aircraftzone.forbidTx,NO_TX_FREQUENCY), Analog, 25 kHz, High power — matching the existing airband channel pattern.AirportSource) — the "Add common aircraft frequencies" checkbox is now available in both zone-grouping modes. When enabled, a select/deselect list of all 16 frequencies is shown (all selected by default) with Select All / Deselect All buttons, and only the selected subset is added.airportChannels.test.ts:AirportsAircraftzone in individual modeTesting
npx vitest run tests/unit/airportChannels.test.ts→ 17 passing.KLAXzone plus a separate 14-channelAircraftzone.