Skip to content

Implement nearby searches for car and bicycle parking - #4165

Merged
leonardehrenfried merged 6 commits into
opentripplanner:dev-2.xfrom
ibi-group:nearest-parking
May 20, 2022
Merged

Implement nearby searches for car and bicycle parking#4165
leonardehrenfried merged 6 commits into
opentripplanner:dev-2.xfrom
ibi-group:nearest-parking

Conversation

@leonardehrenfried

@leonardehrenfried leonardehrenfried commented May 19, 2022

Copy link
Copy Markdown
Member

Summary

When using the nearest GraphQL endpoint CAR_PARK and BICYCLE_PARK was not implemented yet. This PR changes that.

Other changes:

  • Rename VehicleRentalStationVertex to VehicleRentalPlaceVertex
  • Rename BIKE_RENT to VEHICLE_RENT but kept it backwards-compatible in the API
  • Make PlaceAtDistance a record
  • Convert some tests from Junit 4 to Junit 5

Issue

Relates to #4105

Unit tests

Added.

Documentation

Changed the GraphQL docs.

/cc @miles-grant-ibigroup

@leonardehrenfried
leonardehrenfried requested a review from a team as a code owner May 19, 2022 10:02
@leonardehrenfried leonardehrenfried added !Improvement A functional improvement or micro feature IBI test labels May 19, 2022

@optionsome optionsome left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to work. While creating the new API, we should probably reconsider how these filters for vehicle parkings/rentals should work.

@leonardehrenfried
leonardehrenfried merged commit 48de7c4 into opentripplanner:dev-2.x May 20, 2022
@leonardehrenfried
leonardehrenfried deleted the nearest-parking branch May 20, 2022 14:02
t2gran pushed a commit that referenced this pull request May 20, 2022
@leonardehrenfried leonardehrenfried added IBI Developed by or important for IBI Group and removed IBI test labels Jun 21, 2022
@t2gran t2gran added this to the 2.2 milestone Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IBI Developed by or important for IBI Group !Improvement A functional improvement or micro feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants