CJNE / ha-porscheconnect

Porsche Connect custom component for Home Assistant
MIT License
28 stars 11 forks source link

New error #218

Closed woozar closed 9 months ago

woozar commented 9 months ago

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 commented 9 months ago

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 commented 9 months ago

Can you please try #217 ?

woozar commented 9 months ago

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 commented 9 months ago

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 commented 9 months ago

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

vidartorkilseng commented 9 months ago

217 fixed the problem, thanks.

woozar commented 9 months ago

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

CJNE commented 9 months ago

Yes, merging now, thanks @fredriklj !

woozar commented 9 months ago

fixed