-
Notifications
You must be signed in to change notification settings - Fork 69
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
Fixes for Python3, switched to pcryptodome #28
base: master
Are you sure you want to change the base?
Conversation
Thanks @BenWoodford, fine by me. 👍 |
I like it. Thanks @BenWoodford |
In case of a long time since requesting an update from the car the Nissan servers return just `{ "status": 200 }`. Fix to avoid crash in this instance.
Pinging @jdhorne as I don't think PRs fire a Github notification |
Hi @BenWoodford; does this break compatibility with Python 2? I know some folks are still using that. Just want to make sure I know what I'm doing by approving. |
Honestly? No idea. I wasn't aware people were even actively using Python 2 for new projects. I would assume so. This is mostly to fix stuff like tabs being used instead of spaces, etc. However I don't know if |
Fix 25 July API changes.
Updated travellist to response
Version 2.10
This includes numerous fixes for Python3 from @tomhillable and @filcole, which I've merged (sometimes manually...) into my own fork - but I feel like I should PR them. I hope the others don't mind, but right now this lib doesn't actually work with later versions of Python.