Skip to content
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

Review/nav 87 include refactorings from nav 83 review #47

Merged
merged 70 commits into from
Jun 4, 2024

Conversation

clukas1
Copy link
Member

@clukas1 clukas1 commented Jun 4, 2024

This time the correct branch

clukas1 and others added 30 commits May 28, 2024 00:20
Tests and maybe performance improvements coming soon.
- Allows search for STARTS_WITH, ENDS_WITH, CONTAINS and EXACT.
- Duplicate search keys are not yet supported in the index.
- Adjust test builder accordingly.
- Implementation of the public transit service interface.
- Apply visitor pattern to iterate through legs.
- Split service into schedule info and connection routing interface, supporting interface segregation.
- Add trip to stop and fix recursion.
- Extend visitor pattern.
- Rename isoline to getIsolines.
- Setup dependency injection.
- Setup logging and dependencies.
- Add DTO mapper and map objects in controllers.
- Add application.properties for configuration of the service.
- Rename model to dto.
- A walk from a location to a stop, the first leg of a connection.
- A walk from a stop to a location, the last leg of a connection.
- A walk between two locations, the only leg of a connection.
…lementation-based-on-gtfs-and-raptor' into NAV-84-Allow-querying-connections-from-location-or-stop-id-in-REST-API
…s-from-location-or-stop-id-in-REST-API

Nav 84 allow querying connections from location or stop id in rest api
- Adjust spring service.
- Update type mapper and dto mapper.
…o feature/NAV-83-reduce-gtfs-switzerland-to-trams-in-zurich
…ervice

REFACTOR: NAV-81 - Move spatial index from GTFS to service
…fs-and-raptor' into feature/NAV-83-reduce-gtfs-switzerland-to-trams-in-zurich
munterfi and others added 26 commits June 2, 2024 21:26
…fs-and-raptor' into NAV-87-Add-support-to-route-from-parent-stop-to-parent-stop

# Conflicts:
#	src/main/java/ch/naviqore/service/impl/PublicTransitServiceImpl.java
…stop, coordinate-stop, coordinate-coordinate)
…stop' into review/NAV-87-include-refactorings-fromNAV-83-review

# Conflicts:
#	src/main/java/ch/naviqore/service/impl/PublicTransitServiceImpl.java
@munterfi munterfi self-requested a review June 4, 2024 16:21
Copy link
Member

@munterfi munterfi left a comment

Choose a reason for hiding this comment

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

As reviewed

@munterfi munterfi merged commit 1ee976e into main Jun 4, 2024
2 checks passed
@munterfi munterfi deleted the review/NAV-87-include-refactorings-fromNAV-83-review branch June 4, 2024 16:22
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.

3 participants