-
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
Make Transmodel GraphQl API an official OTP API #5573
Make Transmodel GraphQl API an official OTP API #5573
Conversation
src/ext/java/org/opentripplanner/ext/transmodelapi/mapping/PreferencesMapper.java
Outdated
Show resolved
Hide resolved
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #5573 +/- ##
=============================================
- Coverage 67.35% 67.24% -0.12%
+ Complexity 16150 16084 -66
=============================================
Files 1857 1852 -5
Lines 71075 70970 -105
Branches 7402 7399 -3
=============================================
- Hits 47872 47722 -150
- Misses 20745 20782 +37
- Partials 2458 2466 +8 β View full report in Codecov by Sentry. |
src/main/java/org/opentripplanner/apis/transmodel/model/framework/PenaltyForStreetModeType.java
Show resolved
Hide resolved
src/main/java/org/opentripplanner/framework/application/OTPFeature.java
Outdated
Show resolved
Hide resolved
Please resolve the conflicts |
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 agree with Johan's spelling requests but the rest of the code looks good.
Please also resolve the conflicts. |
Co-authored-by: Johan Torin <jtorin@users.noreply.github.com>
src/main/java/org/opentripplanner/framework/application/OTPFeature.java
Outdated
Show resolved
Hide resolved
Sorry, found two typos. |
c8bc65f
to
13d9717
Compare
Co-authored-by: Johan Torin <jtorin@users.noreply.github.com>
Summary
Move Transmodel API out of the Sandbox and make it on by default.
Issue
π₯ No
Unit tests
π₯ No
Documentation
β Doc is updated. We probably need to write better doc on the transmodel API - that is not included here. I have only updated the doc so it should be correct.
Changelog
β
Bumping the serialization version id
π₯ Not needed.