Skip to content

Staging#819

Merged
accius merged 2 commits intomainfrom
Staging
Mar 24, 2026
Merged

Staging#819
accius merged 2 commits intomainfrom
Staging

Conversation

@accius
Copy link
Owner

@accius accius commented Mar 24, 2026

What does this PR do?

Type of change

  • Bug fix
  • New feature
  • Performance improvement
  • Refactor / code cleanup
  • Documentation
  • Translation
  • Map layer plugin

How to test

Checklist

  • App loads without console errors
  • Tested in Dark, Light, and Retro themes
  • Responsive at different screen sizes (desktop + mobile)
  • If touching server.js: caches have TTLs and size caps (we serve 2,000+ concurrent users)
  • If adding an API route: includes caching and error handling
  • If adding a panel: wired into Modern, Classic, and Dockable layouts
  • No hardcoded colors — uses CSS variables (var(--accent-cyan), etc.)
  • No .bak, .old, console.log debug lines, or test scripts included

Screenshots (if visual change)

accius and others added 2 commits March 23, 2026 20:47
rbn.js called gridToLatLonSimple() which only existed in
pskreporter.js. Imported gridToLatLon from shared utils/grid.js
and replaced the undefined references. This was spamming the
error log every few minutes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CTY.DAT (community-maintained DXCC database) has exact callsign
overrides for DXpeditions with precise entity coordinates, but it
was consulted AFTER the static prefix grid table. The prefix table
would match a short prefix (e.g. T → East Africa) before CTY.DAT
could return the correct entity (TX5 → Austral Islands).

Reordered: CTY.DAT is now checked first. The static prefix grid
table is a fallback for when CTY.DAT hasn't loaded or doesn't
have a match.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@accius accius merged commit 51e70a3 into main Mar 24, 2026
3 of 6 checks passed
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