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.

Sign up for GitHub

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

Jump to bottom

Status no maiden #142

Merged
merged 2 commits into from
Jun 25, 2023
Merged

Status no maiden #142

merged 2 commits into from
Jun 25, 2023

Conversation

CBielstein
Copy link
Owner

Description

Implements encode and decode of Status info fields without a position.
Resolves #88.

Changes

  • Implements encode and decode of non-maidenhead status reports
  • Adds required helpers and regex strings
  • Expands tests for new scenarios

Validation

  • Tests and lint are successful
  • Able to run APRSsharp.exe and view status reports without position both with and without timestamp using -f t/s

@CBielstein CBielstein added the enhancement New feature or request label Jun 25, 2023
@CBielstein CBielstein self-assigned this Jun 25, 2023
@CBielstein CBielstein merged commit e9be729 into main Jun 25, 2023
3 checks passed
@CBielstein CBielstein deleted the status-no-maiden branch June 25, 2023 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eddywine eddywine Awaiting requested review from eddywine eddywine is a code owner

Assignees

@CBielstein CBielstein

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Handle Status report without maidenhead
1 participant
@CBielstein