Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARRL DX - Default exchange is wrong (should be 5NN ON) #242

Closed
w7sst opened this issue Feb 13, 2024 · 0 comments · Fixed by #244
Closed

ARRL DX - Default exchange is wrong (should be 5NN ON) #242

w7sst opened this issue Feb 13, 2024 · 0 comments · Fixed by #244
Assignees
Labels
bug Something isn't working verified Bug is fixed and verified

Comments

@w7sst
Copy link
Owner

w7sst commented Feb 13, 2024

Description

An email from Larry, @f6fvy ...

I found by chance that the default exchange for the ARRL DX contest is set to 5NN GH

UserExchangeTbl[scArrlDx] := ReadString(SEC_STN, 'ArrlDxExchange', '5NN GH');
.
Actually, GH is an ARRL/RAC section, and the ARRL DX requires a State or a Province. So, for VE3NEA, the default exchange must be 5NN ON Ontario). If you set GH, MRCE will complain it's not valid.

73
Larry - F6FVY

Looks like I need to change a default string in a table.

@w7sst w7sst added the bug Something isn't working label Feb 13, 2024
@w7sst w7sst self-assigned this Feb 13, 2024
w7sst added a commit that referenced this issue Feb 14, 2024
- convert to table-driven system to reduce future errors.
- NAQP default exchange changed from '5NN GH' to '5NN ON'
- updated several default exchange settings (usually based on contest author/qth)
@w7sst w7sst linked a pull request Feb 14, 2024 that will close this issue
@w7sst w7sst added the fixed - ready for validation bug is fixed; ready for validation label Feb 14, 2024
w7sst added a commit that referenced this issue Feb 14, 2024
- convert to table-driven system to reduce future errors.
- NAQP default exchange changed from '5NN GH' to '5NN ON'
- updated several default exchange settings (usually based on contest
author's name/QTH)
@w7sst w7sst added verified Bug is fixed and verified and removed fixed - ready for validation bug is fixed; ready for validation labels Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working verified Bug is fixed and verified
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant