search

guerrerotook / securitas-direct-new-api

This repository contains the new securitas direct API that can be integrated in Home Assistant
Apache License 2.0
79 stars 30 forks source link

Missing Await for new AsyncForwardEntrySetups #108

Closed SirLouen closed 1 year ago

SirLouen commented 1 year ago

Solving issue #107

guerrerotook commented 1 year ago

I'm working on that right now, but there is more that just adding the await. Apparently in the config?flow is expecting a ConfigEntry and not a OrderedDict so that will fail also. Let's pause the PR until I fixed the issue. But thanks for the efford!

SirLouen commented 1 year ago

I'm working on that right now, but there is more that just adding the await. Apparently in the config?flow is expecting a ConfigEntry and not a OrderedDict so that will fail also. Let's pause the PR until I fixed the issue. But thanks for the efford!

It works for me though, but It's true that there was a warning in the type of the PLATFORMS the last time I checked. Still the panel was working by just introducing await, so it requires further research to solve it.

guerrerotook commented 1 year ago

The issue is for when adding the integration for the first time. I have the new code ready.