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

Issue with Enyaq last_connected parsing #70

Closed
seriousch opened this issue Feb 13, 2023 · 1 comment · Fixed by #73
Closed

Issue with Enyaq last_connected parsing #70

seriousch opened this issue Feb 13, 2023 · 1 comment · Fixed by #73

Comments

@seriousch
Copy link

seriousch commented Feb 13, 2023

Hi,

I'm (indirectly) using skodaconnect from Home Assistant. Many thanks for the work by the way, it's great to have this library.

With Enyaq, there is this recurrent issue happening quite often:

File "/usr/local/lib/python3.10/site-packages/skodaconnect/dashboard.py", line 127, in state val = super().state File "/usr/local/lib/python3.10/site-packages/skodaconnect/dashboard.py", line 58, in state if hasattr(self.vehicle, self.attr): File "/usr/local/lib/python3.10/site-packages/skodaconnect/vehicle.py", line 1542, in last_connected last_connected = datetime.strptime(last_connected_utc,'%Y-%m-%dT%H:%M:%S.%fZ').astimezone().replace(tzinfo=None).replace(microsecond=0) TypeError: strptime() argument 1 must be str, not None

Thanks!

dvx76 added a commit to dvx76/skodaconnect that referenced this issue Mar 20, 2023
Fixes skodaconnect#70

```
File "/usr/local/lib/python3.10/site-packages/skodaconnect/dashboard.py", line 127, in state val = super().state
File "/usr/local/lib/python3.10/site-packages/skodaconnect/dashboard.py", line 58, in state if hasattr(self.vehicle, self.attr):
File "/usr/local/lib/python3.10/site-packages/skodaconnect/vehicle.py", line 1542, in last_connected last_connected = datetime.strptime(last_connected_utc,'%Y-%m-%dT%H:%M:%S.%fZ').astimezone().replace(tzinfo=None).replace(microsecond=0) TypeError: strptime() argument 1 must be str, not None
```
@seriousch
Copy link
Author

Many thanks @dvx76 and @Farfar

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 a pull request may close this issue.

1 participant