Skip to content

making route display a little more compact - #547

Merged
makrsmark merged 1 commit into
airframesio:masterfrom
makrsmark:bugfix/h1-76401
Oct 24, 2026
Merged

makrsmark merged 1 commit into
airframesio:masterfrom
makrsmark:bugfix/h1-76401

Conversation

@makrsmark

@makrsmark makrsmark commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Bug Fixes
    • Waypoint names now preserve the coordinate text as it appeared in the original message.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

The waypoint decoder now sets waypoint names from the original 13-character coordinate text in the message, instead of the normalized coordinate string.

Changes

Waypoint Coordinate Text

Layer / File(s) Summary
Set waypoint name from input coordinates
lib/plugins/Label_H1_76401.ts
Waypoint names now use the original 13-character coordinate substring from the message.

Priority: Not assessed

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Possibly related PRs

Merge Risk: Moderate · up to b6387

Spaced-coordinate H1 messages can produce incorrect route waypoint data; fix the field-width handling before merging.

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit reads the coordinates clear
Thirteen characters stay near
No normalized string takes their place
The waypoint keeps its source-text trace
Hop, hop, names retain their face

Comment @coderabbitai help to get the list of available commands.

@makrsmark
makrsmark merged commit d4fa50d into airframesio:master Oct 24, 2026
4 of 5 checks passed
@makrsmark
makrsmark deleted the bugfix/h1-76401 branch October 24, 2026 12:43
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.

1 participant