Unable to configure grocy custom component

ccmy1226 commented 1 year ago

Unless all relevant information is provided, I can't help you

After filling in the form for configuring the integration, a simple error message: "Unknown error occurred" is returned.

Expected behavior

Integration is configured and sensors are available

To Reproduce

Steps to reproduce the behavior:

Go to Configuration > Integrations Click on "+ add integration" and select Grocy Fill in the required details and click "Submit" See error

General information to help debugging:

What sensors do you have enabled? Are they working and/or what state are they in? Do you have the corresponding functions enabled in Grocy?

Nothing is enabled yet

What is your installed versions of Home Assistant, Grocy and this integration? Home assistant Core 2022.9.4 Supervisor 2022.08.6 Host 8.5 Grocy 0.18.2 Custom Grocy: 4.5.2 installed via HACS

How do you have Grocy installed? Add-on or external? Add-on

Have you added debugging to the log, and what does the log say? This error originated from a custom integration.

Logger: custom_components.grocy.config_flow Source: custom_components/grocy/config_flow.py:42 Integration: Grocy (documentation, issues) First occurred: 11:19:25 PM (2 occurrences) Last logged: 11:25:33 PM

Expecting value: line 1 column 1 (char 0)

JSON service data (if related to using a service)

Additional context I tried all combination in the configuration panel, all led to the same result. Localhost and IP address fails right away, my external dns host spins for a while and errors out as well

adaschma commented 1 year ago

Hi ccmy1226!

The TL;DR is upgrade your grocy to 3.3 should fix the problem.

Your grocy version is very old. Your version is 0.18.2 while 3.3.1 is the current version. While I haven't tested it with a grocy that old, I can reproduce your problem with a grocy 3.1:

Logger: custom_components.grocy.config_flow Source: custom_components/grocy/config_flow.py:42 Integration: Grocy (documentation, issues) First occurred: 09:30:39 (1 occurrences) Last logged: 09:30:39

2 validation errors for SystemInfoDto os field required (type=value_error.missing) client field required (type=value_error.missing)

Hope that helps. --adaschma

grablair commented 1 year ago

Hi @adamschmalhofer, the 0.18.2 version that @ccmy1226 was talking about is the HA Addon version. The underlying Grocy version is different, however.

For example, the latest version at the time of writing is 0.18.3, but the underlying Grocy version is 3.3.2.

grablair commented 1 year ago

@ccmy1226 , I was having the same issue. When I followed these instructions:

If you have configured the Grocy add-on as described, use port 9192 (without https).

And I had the following values entered:

Grocy API URL: http://<internal-ip> (note `http`, as documented in the README)
Grocy API Key: <key>
Port Number: 9192
Verify SSL: No

It would fail with the same error as you. When I submitted it with https instead of http (against what the README specifies), then it succeeded. 🤷🏻‍♂️

custom-components / grocy