CBielstein / APRSsharp

APRS# - Modern APRS software for the amateur radio community
MIT License
12 stars 5 forks source link

AprsSharp can display decoded APRS-IS packets #80

Closed CBielstein closed 2 years ago

CBielstein commented 3 years ago

Description

Currently, AprsSharp command line app prints out raw TNC2-encoded APRS packets. This is not very intuitive for the reader as most humans are not familiar with TNC2 and APRS encodings.

For this issue:

Since different APRS packet types have different relevant fields, there may be some work here to decide which fields are relevant. It's also okay to take an easy approach and refine later e.g printing all fields or printing specific common fields (e.g. callsign, comment, timestamp?, etc.).

Acceptance Criteria

CBielstein commented 2 years ago

This issue is NOT about decoding all possible APRS packets. That is covered by Milestone 7. The point of this issue is to hook up better UI for printing out packets which we CAN decode. Once that's done, the issues in Milestone 7 will increase the packets we can decode.

So this issue should focus on UI in the console app and printing out human-readable information, which we can expand upon later.

CBielstein commented 2 years ago

Basic UI print logic is wired up in the packet refactor as part of #93, but it is likely still worthwhile to clean up the UI output here to be much cleaner.

We'll also definitely want to add an event to pass successfully decoded packets up to the client and switch Program.cs to use that instead of forcing all clients to handle decode logic directly.

CBielstein commented 2 years ago

I split the code-cleanliness part of this (as mentioned in previous comments) off in to #103 as #102 will actually resolve the original acceptance criteria of this issue.