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

New error #218

Closed
woozar opened this issue Nov 26, 2023 · 9 comments
Closed

New error #218

woozar opened this issue Nov 26, 2023 · 9 comments

Comments

@woozar
Copy link

woozar commented Nov 26, 2023

I have got a new error (at least for me it is new)

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 290, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/porscheconnect/init.py", line 215, in _async_update_data
vehicle["services"] = await self.controller.getServices(vin)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/pyporscheconnectapi/client.py", line 488, in getServices
is_privacy = data['vehicleServiceEnabledMap']['VSR'] == 'DISABLED'

KeyError: 'vehicleServiceEnabledMap'
@vidartorkilseng
Copy link

Integration has worked ok, but now the error below.

Unexpected error fetching porscheconnect data: 'vehicleServiceEnabledMap'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 290, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/porscheconnect/init.py", line 215, in _async_update_data
vehicle["services"] = await self.controller.getServices(vin)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/pyporscheconnectapi/client.py", line 488, in getServices
is_privacy = data['vehicleServiceEnabledMap']['VSR'] == 'DISABLED'
~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'vehicleServiceEnabledMap'

@fredriklj
Copy link
Collaborator

Can you please try #217 ?

@woozar
Copy link
Author

woozar commented Nov 28, 2023

Can you please try #217 ?

I do not really understand what you mean. Is this an update? If so, my HACS does not offer it to me. I am up to date with 0.0.17

Edit: opening it on my laptop (instead of smartphone) helped. It is a pull request, but it seems not to be merged, yet. I will need to spend some time to figure out, how to install it on my home assistant. (I never tried a manual installation, so far)

@woozar
Copy link
Author

woozar commented Nov 28, 2023

TLDR: It works!

It took me a while to figure out, how to scp the correct files to the correct location (I was not even able to checkout your branch, but I was able to reproduce your changes) and after restarting home assistant like 10 times and deleting and recreating my login, it finally works again. Thanks <3
I will leave the ticket open, until it is merged and available through HACS.

@jem
Copy link

jem commented Nov 28, 2023

I had a similar error, and #217 fixed it for me, too.

@vidartorkilseng
Copy link

#217 fixed the problem, thanks.

@woozar
Copy link
Author

woozar commented Dec 1, 2023

@fredriklj, @CJNE will #217 be merged any time soon?

@CJNE
Copy link
Owner

CJNE commented Dec 2, 2023

Yes, merging now, thanks @fredriklj !

@woozar
Copy link
Author

woozar commented Dec 3, 2023

fixed

@woozar woozar closed this as completed Dec 3, 2023
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

No branches or pull requests

5 participants