Work through config_flow issues and warnings fixes #6 fixes #8 fixes #15

custom_components/kostal_piko/config_flow.py

@@ -1,4 +1,5 @@
 """Config flow for Kostal Piko solar inverters."""
+
 import logging
 from typing import Any
 
@@ -28,8 +29,12 @@ async def test_connection(
     hass: HomeAssistant, data: dict[str, Any]
 ) -> tuple[str, str, str]:
     """Tests the connection to the inverter and returns its name."""
+    host = data["host"]
+    username = data["username"]
+    password = data["password"]
+
     session = async_get_clientsession(hass)
-    inverter = kostal.Piko(session, data["host"], data["username"], data["password"])
+    inverter = kostal.Piko(session, host, username, password)
     res = await inverter.fetch_props(
         kostal.SettingsGeneral.INVERTER_NAME,
         kostal.SettingsGeneral.INVERTER_MAKE,
@@ -60,12 +65,26 @@ async def async_step_user(
 
         if user_input is not None:
             try:
-                name, make, serial = await test_connection(self.hass, user_input)
+                host = user_input["host"]
+                username = user_input["username"]
+                password = user_input["password"]
+
+                # Opportunistically try to add http:// if neither http:// nor https:// is specified
+                if not host.startswith("http"):
+                    host = "http://" + str(host)
+
+                # host cannot end with / since more path parameters are added to it by the library
+                if host.endswith("/"):
+                    host = host[:-1]
+
+                setup_input = {"host": host, "username": username, "password": password}
+
+                name, make, serial = await test_connection(self.hass, setup_input)
                 await self.async_set_unique_id(serial)
                 self._abort_if_unique_id_configured()
 
                 return self.async_create_entry(
-                    title=f"{make} {name} ({serial})", data=user_input
+                    title=f"{make} {name} ({serial})", data=setup_input
                 )
 
             except ValueError as err:

custom_components/kostal_piko/const.py

@@ -1,4 +1,5 @@
 """Constants for the Kostal Piko integration."""
+
 from __future__ import annotations
 
 from collections.abc import Callable
@@ -35,7 +36,7 @@
     3: "feed_in",
     4: "feed_in_limited",
     7: "insulation_measurement",
-    8: "waiting_time"
+    8: "waiting_time",
 }
 
 

custom_components/kostal_piko/manifest.json

@@ -8,5 +8,5 @@
   "issue_tracker": "https://github.com/scheidtdav/homeassistant-kostal-piko/issues",
   "iot_class": "local_polling",
   "requirements": ["pykostal==0.0.2"],
-  "version": "24.01.0"
+  "version": "24.12.0"
 }

custom_components/kostal_piko/translations/de.json

@@ -9,9 +9,14 @@
     "step": {
       "user": {
         "data": {
-          "host": "Host",
+          "host": "Hostname oder IP Adresse",
           "username": "Benutzername",
           "password": "Passwort"
+        },
+        "data_description": {
+          "host": "Der Hostname oder die IP-Adresse (inkl. http:// oder https://) des Wechselrichters.",
+          "username": "Der Benutzername mit dem sich auf der Webseite des Wechselrichters eingeloggt wird.",
+          "password": "Das Passwort mit dem sich auf der Webseite des Wechselrichters eingeloggt wird."
         }
       }
     },

custom_components/kostal_piko/strings.json → custom_components/kostal_piko/translations/en.json

@@ -9,9 +9,14 @@
     "step": {
       "user": {
         "data": {
-          "host": "Host",
+          "host": "Host or IP",
           "username": "Username",
           "password": "Password"
+        },
+        "data_description": {
+          "host": "The hostname or IP address (including http:// or https://) of the inverter.",
+          "username": "The username used to log into the inverters webpage.",
+          "password": "The password used to log into the inverters webpage."
         }
       }
     },