-
Notifications
You must be signed in to change notification settings - Fork 1k
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.
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
Refetch transit leg with a leg query of GTFS GraphQL API #6045
Merged
vesameskanen
merged 29 commits into
opentripplanner:dev-2.x
from
HSLdevcom:gfts-leg-node-refetch
Sep 27, 2024
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
5e18c0a
GTFS graphql API supports querying itinerary leg status
vesameskanen 021e07c
Catch non-transit legs properly
vesameskanen 01b460d
Add type resolver for Leg
vesameskanen 2182066
Fix formatting
vesameskanen 7e2033f
Do not encode null reference as 'null' string
vesameskanen 6b92741
More accurate docs about leg query
vesameskanen e489f05
Add leg id to plan query test
vesameskanen 6bd5d04
Use self-documenting value for non-transit leg ids
vesameskanen a998e52
Add more docs about leg re-fetching into gtfs graphql schema
vesameskanen cf37e89
Support querying leg's realTimeState in GTFS graphql api
vesameskanen cd54cbe
Query realtimeState in extended plan test
vesameskanen 4025d5b
Revert LegReferenceSerializer changes
vesameskanen b206ce1
Generate unique id for non-transit legs
vesameskanen 15f1f34
Restore LegReferenceSerializer null test
vesameskanen 20f855a
Update plan-extended expoctations to use unique leg ids
vesameskanen 9b3c4d5
Revert "Generate unique id for non-transit legs"
vesameskanen 14fd13e
Remove node interface from LegType and add leg query
vesameskanen 207cd5d
Merge remote-tracking branch 'otp/dev-2.x' into gfts-leg-node-refetch
vesameskanen 71a6315
Update auto generated files
vesameskanen cd660b2
Fix schema formatting
vesameskanen 97d2180
Fix leg id access
vesameskanen b75c047
Remove leg resolver from node type
vesameskanen 1a43221
Use ID parameter type for leg query
vesameskanen 58b382f
Update src/main/resources/org/opentripplanner/apis/gtfs/schema.graphqls
vesameskanen f6a8d8f
Suggested review changes
vesameskanen 71caaa5
Merge remote-tracking branch 'origin/gfts-leg-node-refetch' into gfts…
vesameskanen ba100e8
Use string type instead of relay ID type for transit leg identification
vesameskanen 45d6f98
Update src/main/java/org/opentripplanner/apis/gtfs/datafetchers/LegIm…
vesameskanen 857b815
Update src/main/resources/org/opentripplanner/apis/gtfs/schema.graphqls
vesameskanen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -151,6 +151,8 @@ | |
arrival | ||
} | ||
accessibilityScore | ||
id | ||
realtimeState | ||
} | ||
} | ||
} | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just realised that this doesn't use a proper mapper, which I think is a very useful thing.
Since I already approved, I wrote the code myself: leonardehrenfried@a763bc5
If you want you can cherry pick it into this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, too late. I will open another one myself.