Skip to content

Skip ATIS and tower view callsigns in parsing Slurper API response - #106

Merged
AussieScorcher merged 5 commits into
stopbars:mainfrom
19wintersp:patch-1
Oct 3, 2026
Merged

AussieScorcher merged 5 commits into
stopbars:mainfrom
19wintersp:patch-1

Conversation

@19wintersp

@19wintersp 19wintersp commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Slurper sometimes returns ATIS connections before ATC connections. ATIS connections are not relevant to us.

Changes Made

  • Skip _ATIS callsigns when parsing Slurper response
  • Skip _TV callsigns when parsing Slurper response

Checklist:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Changes how the system parses pilot connection data from the VATSIM API.

The PR appears safe to merge; no outstanding findings remain.

Summary

The PR changes Slurper response parsing to skip ATIS and tower-view callsigns before selecting a connection. The latest change corrects the suffix values so that filtering takes effect.

Reviews (3) · Last reviewed commit: "fair enough"

Comment thread src/services/vatsim.ts Outdated
@19wintersp 19wintersp changed the title Skip ATIS callsigns in parsing Slurper API response Skip ATIS and tower view callsigns in parsing Slurper API response Oct 1, 2026
Comment thread src/services/vatsim.ts
@AussieScorcher
AussieScorcher merged commit a01fd27 into stopbars:main Oct 3, 2026
1 check passed
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.

2 participants