Timeout improve error message about deebot-4-home-assistant HOT 5 OPEN

As explained in the comment above, you are referring to the api communication between the client and HA instance.

I can try to see if I can improve the error handling, but the timeout happens after 1-2 minutes and I don't know if HA allows me to give feedback that late.
The timeout is controlled by ecovacs, so I can't change it.

from deebot-4-home-assistant.

After deleting the app, resetting my Deboot T8+ and setting it up again from scratch, both the app and the commands are now working.

But it would still be great to get a proper error response if this happens again 🙏

from deebot-4-home-assistant.

wait for response timed out means that the ecovacs server did not get a response from your robot in time also you see this in the ecovacs app. You need to check if your robot is online and has a stable connection.

I checked the websocket messages and I noticed it took a really long time to get a response, but when I did get a response it's like this:

{
    "id": 13,
    "type": "result",
    "success": true,
    "result": {
        "context": {
            "id": "01811a2376985661bd8d5344471ea557",
            "parent_id": null,
            "user_id": "xxx"
        }
    }
}

This has nothing to do with the integration. This websocket connection is made between your client (browser, app) and your home assistant instance. As you can reach your instance, you will get also a response.

But it would still be great to get a proper error response if this happens again

The state is set to unknown, as we cannot identify and a warning is logged. What error message would you expect?

from deebot-4-home-assistant.

@edenhaus If an integration times out, I would at minimum expect to see a success: false in response to the call_service message.

in an ideal world, a success: false, accompanied by an error message in the result saying it timed out.

something like:

{
    "id": 13,
    "type": "result",
    "success": false,
    "result": {
         "error": {
            code: 500,
            message: "wait for response timed out",
         },
        "context": {
            "id": "01811a2376985661bd8d5344471ea557",
            "parent_id": null,
            "user_id": "xxx"
        }
    }
}

Although I am not familiar with creating integrations, so maybe that is not even possible? But it was very frustrating debugging this and seeing that home assistant says my command was successful when it wasn't leaving my state out of sync with reality.

from deebot-4-home-assistant.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from deebot-4-home-assistant.