aprs-parser-rs icon indicating copy to clipboard operation
aprs-parser-rs copied to clipboard

APRS Messages

Open scd31 opened this issue 3 years ago • 4 comments

Adds support for APRS messages.

Also did some minor refactoring(renamed `APRS**** to Aprs****, Renamed APRSMessage to AprsPacket to make room for the new struct)

scd31 avatar Dec 29 '21 17:12 scd31

can you split the formatting and refactoring changes from the feature implementation? mixing everything in one commit/PR makes this quite hard to review 🙏

Turbo87 avatar Dec 29 '21 17:12 Turbo87

@scd31 is there a specification for this? do you have a reference/URL/something where I can read up on this?

Turbo87 avatar Jan 03 '22 21:01 Turbo87

@Turbo87 I'm using this: http://www.aprs.org/doc/APRS101.PDF

Messages start on page 71/81.

scd31 avatar Jan 04 '22 23:01 scd31

Are there any plans to merge this? I know it needs the conflicts resolved, which I'm happy to fix. Just wondering if I should fork this instead

scd31 avatar May 26 '22 19:05 scd31

sorry for the long silence. the package is not high on my priority list :-/

Turbo87 avatar Aug 05 '22 10:08 Turbo87

No worries! Just glad it got merged.

scd31 avatar Aug 05 '22 10:08 scd31