fix(new-contact): refresh path and QRZ panel on every callsign search#211
Open
patrickrb wants to merge 1 commit into
Open
fix(new-contact): refresh path and QRZ panel on every callsign search#211patrickrb wants to merge 1 commit into
patrickrb wants to merge 1 commit into
Conversation
Two issues on the Log a contact page when searching a callsign: 1. Stale data persisted across searches. The lookup handler merged results with `prev` (`data.grid_locator || prev.gridLocator`), so when a new operator's QRZ record returned no grid, the previous operator's grid (and lat/lng) silently survived and the path map never moved. The callsign field also held the old name/qth/grid while the next lookup was in flight. Now the lookup-derived fields reset on every callsign change and are replaced (not merged) when the new lookup returns. 2. The "verified" panel only displayed initials + name + grid + country + qth, throwing away the rest of QRZ's response. The panel now shows the operator's QRZ profile image (with initials fallback on missing or broken image), nickname, MapPin'd address line, license-class chip (Extra/General/Tech/Club/...), LoTW/eQSL participation chips, QSL via, and a View on QRZ link. lib/qrz.ts now parses image, nickname, aliases, city, state, class, lotw, eqsl from the XML. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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
Two fixes on the Log a contact page when looking up a callsign:
data.grid_locator || prev.gridLocator), so when the new operator's QRZ record didn't carry a grid the previous operator's grid (and lat/lng) silently survived — the path map never moved. The callsign field also held the old name/qth/grid while the next lookup was in flight. Lookup-derived fields now reset on every callsign change and are replaced (not merged) when the new lookup returns.lib/qrz.tsnow parsesimage,nickname,aliases,city,state,class,lotw,eqslfrom the XML.Test plan
W1AW→FN31pr), then change to one with no grid → grid input + path map clear instead of holding the previous oneqrz.com/db/<CALL>in a new tab🤖 Generated with Claude Code