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

Not working for eUP! after migrated to the new Volkswagen API #227

Open
AlexJacu opened this issue Nov 23, 2023 · 2 comments
Open

Not working for eUP! after migrated to the new Volkswagen API #227

AlexJacu opened this issue Nov 23, 2023 · 2 comments

Comments

@AlexJacu
Copy link

Is your feature request related to a problem? Please describe.
After closing old API and migrate eUP! to new API the eUP is not working using this integration

@wherzig
Copy link

wherzig commented Aug 17, 2024

You may try what I did today:

volkswagen_we_connect_id # diff -u __init__.py.old __init__.py
--- __init__.py.old	2024-08-17 12:51:01.000000000 +0200
+++ __init__.py	2024-08-17 17:03:49.029374797 +0200
@@ -33,7 +33,7 @@
 
 _LOGGER = logging.getLogger(__name__)
 
-SUPPORTED_VEHICLES = ["ID.3", "ID.4", "ID.5", "ID. Buzz", "ID.7 Limousine"]
+SUPPORTED_VEHICLES = ["ID.3", "ID.4", "ID.5", "ID. Buzz", "ID.7 Limousine", "e-up!"]
 
 
 @dataclass
@@ -67,6 +67,8 @@
         for vin, vehicle in _we_connect.vehicles.items():
             if vehicle.model.value in SUPPORTED_VEHICLES:
                 vehicles.append(vehicle)
+            else:
+                _LOGGER.error("UNSUPPORTED vehicle '%s'. Skipped ", vehicle.model.value)
 
         domain_entry: DomainEntry = hass.data[DOMAIN][entry.entry_id]
         domain_entry.vehicles = vehicles
volkswagen_we_connect_id # diff -u number.py.old number.py
--- number.py.old	2024-08-17 12:51:01.000000000 +0200
+++ number.py	2024-08-17 14:03:03.521105950 +0200
@@ -77,7 +77,7 @@
         return int(
             get_object_value(
                 self.data.domains["charging"]["chargingSettings"].targetSOC_pct.value
-            )
+            ) or 0
         )
 
     async def async_set_native_value(self, value: float) -> None:

Problem was that e-Up does not support self.data.domains["charging"]["chargingSettings"].targetSOC_pct.value .

Right now many of the sensors seem to work:

image
image

Not all Sensors are supported. I did also not test if there is any way to change configuration.

And yes. It is just a quick hack. No warranty!

But for me it seems sufficient.

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

2 participants