-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/nav 145 extract python client package #1
Conversation
- Refactor imports and rename to snake_case. - Fix mypy issues.
- Docker environment with running service. - Integration test cases.
…t need access to class attributes
- Use alias notation to parse correct fields.
- PublicTransitClientExceptions covers APIErrors. - Debug log parameters. - Instead of building the query string manually, pass a dict to requests.
- Why is this soo annoying in python?!? - Just convert it to statically typed language so the compiler can help!!!
- Run black and isort.
Added Docstrings and changed typing to newer approach without imports from typing.
I suggest implementing the changes to the API for the improved query configs in a separate issue. |
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.
Really nice work! Some stuff I can learn from here. Added some more changes, if you agree with those, I think we can merge this one.
… also be between locations not only stops.
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.
All good, ready for lift off.
Public transit service client Python package which works with the service v0.4.0.
Since the
snake_case
notation is used, the public transit viewer would need some adjustments. Also the new query parameters from NAV-131 are not yet supported.