search

mitch-dc / volkswagen_we_connect_id

Apache License 2.0
201 stars 52 forks source link

after updating to latest version of HA getting login error #143

Closed sebking1971 closed 1 year ago

sebking1971 commented 1 year ago

ogger: homeassistant.config_entries Source: custom_components/volkswagen_we_connect_id/init.py:47 Integration: Volkswagen We Connect ID First occurred: 16:30:49 (1 occurrences) Last logged: 16:30:49

Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry await hass.async_add_executor_job(_we_connect.login) File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login self.__session.login() File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 76, in login response = self.doWebAuth(authorizationUrl) File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 242, in doWebAuth raise AuthentificationError('You have to login at myvolkswagen.de and accept the terms and conditions') weconnect.errors.AuthentificationError: You have to login at myvolkswagen.de and accept the terms and conditions

jonasbkarlsson commented 1 year ago

I installed 2023.3.0, but it still has requests==2.28.1. Seems like the PR wasn't included in the 2023.3.0 release. https://github.com/home-assistant/core/blob/2023.3.0/requirements.txt

jonas-wilnerzon commented 1 year ago

@jonasbkarlsson It will probably be included in the april release of HA core. https://github.com/home-assistant/core/pull/88956#issuecomment-1451347982

mitch-dc commented 1 year ago

My hope it is included in the .1 release

ChristianH21220 commented 1 year ago

https://github.com/home-assistant/core/pull/88956#issuecomment-1451662705

Let's wait for .1

viddi87 commented 1 year ago

My hope it is included in the .1 release

It's here 😀

petosak commented 1 year ago

Yes! Thank you very much @mitch-dc !!!!

torge70 commented 1 year ago

Updated to 2023.3.1. Itegration still not working. Removed and added the integration. Some sensors have appeared but not all. There are still errors.

Unexpected exception importing platform custom_components.volkswagen_we_connect_id.sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 779, in get_platform cache[full_name] = self._import_platform(platform_name) File "/usr/src/homeassistant/homeassistant/loader.py", line 796, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") File "/usr/local/lib/python3.10/importlib/init.py", line 126, in import_module return _bootstrap._gcd_import(name[level:], package, level) File "", line 1050, in _gcd_import File "", line 1027, in _find_and_load File "", line 1006, in _find_and_load_unlocked File "", line 688, in _load_unlocked File "", line 883, in exec_module File "", line 241, in _call_with_frames_removed File "/config/custom_components/volkswagen_we_connect_id/sensor.py", line 100, in device_class=SensorDeviceClass.energy, File "/usr/local/lib/python3.10/enum.py", line 437, in getattr raise AttributeError(name) from None AttributeError: energy