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

Add fare product types to types package #614

Conversation

daniel-heppner-ibigroup
Copy link
Contributor

This PR adds the new fare product types to the types package. It breaks compatibility with older versions of OTP and REST API.

Copy link
Collaborator

@miles-grant-ibigroup miles-grant-ibigroup left a comment

Choose a reason for hiding this comment

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

I'm not sure if a refactor commit can be a breaking change, but other than that this works great for me! Thanks for the changes.

@daniel-heppner-ibigroup
Copy link
Contributor Author

don't know how to fix this CI issue... it's failing because the types break stuff in other packages. but if I merge them in the same PR we can't update the version number and so they still fail.

@miles-grant-ibigroup
Copy link
Collaborator

I think the solution is to merge it all, get the ci to pass, merge into alpha, do an alpha release, use those versions here, and then it releases. then a subsequent pr updates the version numbers away from alpha

@miles-grant-ibigroup miles-grant-ibigroup added the BLOCKERS Blockers exist outside of otp-ui (e.g., backend, service, etc...) label Jul 6, 2023
@daniel-heppner-ibigroup daniel-heppner-ibigroup deleted the gql-transition/fare-product-types branch July 20, 2023 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLOCKERS Blockers exist outside of otp-ui (e.g., backend, service, etc...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants