Skip to content

Enhance PSK Reporter spots with default-on setting and map-side band/mode/time filters#89

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/feature-add-pskreporter-reporting
Draft

Enhance PSK Reporter spots with default-on setting and map-side band/mode/time filters#89
Copilot wants to merge 2 commits into
mainfrom
copilot/feature-add-pskreporter-reporting

Conversation

Copy link
Copy Markdown

Copilot AI commented Jun 2, 2026

This updates the in-progress PSK Reporter feature so users can actually work their own spots in-app: reporting/overlay is enabled by default, and map spots can be filtered by band, mode, and time window (similar to PSK Reporter UX).

  • Default behavior and settings integration

    • Set PSK Reporter overlay to default on for new installs.
    • Added a persistent Settings toggle (PSK Reporter Spots) wired to pskOverlayEnabled.
  • Map filtering controls (Band / Mode / Time)

    • Added lightweight filter pills on Map when PSK overlay is enabled:
      • Band: ALL | CUR
      • Mode: ALL | FT8 | FT4
      • Time: 15m | 1h | 6h | 24h
    • Applied filtering on returned spots by:
      • current rig band (CUR),
      • selected mode,
      • report age (flowStartSeconds).
  • PSK Reporter client query flexibility

    • Extended fetchSpotsForMe to accept optional modeFilter.
    • Keeps FT8 default behavior, and omits mode param for “ALL” mode queries.
  • Targeted test coverage updates

    • Added client tests for mode query behavior (mode=FT4 vs omitted mode).
    • Added map filter tests validating combined band/mode/time filtering logic.
val spots = PskReporterClient.fetchSpotsForMe(
    call = call,
    secondsBack = pskTimeFilter.secondsBack,
    modeFilter = pskModeFilter.apiValue, // null => all modes
)

val filtered = filterPskSpots(
    spots = spots,
    bandFilter = pskBandFilter,
    modeFilter = pskModeFilter,
    currentBandHz = GeneralVariables.band,
    maxAgeSeconds = pskTimeFilter.secondsBack,
    nowEpochSeconds = System.currentTimeMillis() / 1000L,
)

Copilot AI linked an issue Jun 2, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add option to report decoded data to pskreporter Enhance PSK Reporter spots with default-on setting and map-side band/mode/time filters Jun 2, 2026
Copilot AI requested a review from patrickrb June 2, 2026 02:05
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.

Feature Request: Add Pskreporter reporting

2 participants