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

314 nil when logging a partial callsign #316

Merged
merged 2 commits into from
Jun 9, 2024

Conversation

w7sst
Copy link
Owner

@w7sst w7sst commented May 31, 2024

After entering a partial callsign and exchange information, the user will hit Enter to finish the QSO and log the QSO. The QSO is not logged.

This was caused by the logging module only comparing for a perfect match against the caller callsign and not considering the common occurrence of a partially-correct callsign. Code was changed to allow a partially-correct callsign. This allows these QSO to be included in the log as expected.

Checkin notes:

Improve handling of partial callsign matches

- Report both callsign and exchange errors in QSO log.
- Add argument to IsMyCall to enable random result for lids.

Issue #315 will add improved messages for the partially-correct callsign case. This next review will be posted soon.

Once the second pull request/review is submitted, I will post an engineering build for testing. Stay tuned...

w7sst added 2 commits May 30, 2024 20:19
- Report both callsign and exchange errors in QSO log.
- Add argument to IsMyCall to enable random result for lids.
@w7sst w7sst requested review from f6fvy, scotthibbs, jr8ppg and K6OK May 31, 2024 07:53
@w7sst w7sst self-assigned this May 31, 2024
Base automatically changed from 200-nil-if-ghosted-before-logging to main June 7, 2024 21:40
@w7sst w7sst merged commit 93d5b92 into main Jun 9, 2024
@w7sst w7sst linked an issue Jun 9, 2024 that may be closed by this pull request
4 tasks
@w7sst w7sst deleted the 314-nil-when-logging-a-partial-callsign branch June 9, 2024 22:01
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.

NIL when logging a partial callsign
1 participant