Skip to content
This repository has been archived by the owner on Oct 11, 2022. It is now read-only.

Config flow not working #37

Closed
anthonyangel opened this issue Nov 20, 2019 · 5 comments
Closed

Config flow not working #37

anthonyangel opened this issue Nov 20, 2019 · 5 comments

Comments

@anthonyangel
Copy link

Hi, thanks for all the recent work which you've put into this.

I'm unable to authenticate via the the new config flow integration, I get the following error when I try to add the Home Connect integration.

Screenshot_20191120-092632

I'm running home assistant 102.0b2, have your component installed via HACS (the current code in the master branch) and have updated the callback URL in the home connect developer site as per the docs.

I'll enable more verbose logging, as I currently have no more useful information to look into this.

@DavidMStraub
Copy link
Owner

Have you been able to install it in HACS without changes? I just tried it out and the INSTALL button is greyed out (this is HACS 0.17.3 on HA 102.0b2).

@DavidMStraub
Copy link
Owner

Nevermind. Thhe initial load of HACS just took very long, then the button appeared.

@DavidMStraub
Copy link
Owner

So, for me it's working, actually.

Can you please try the newest commit? There was an unhandled error that you might have hit.

@anthonyangel
Copy link
Author

This was PEBCAK on my part, I assumed that if I was using the new Config Flow integration then I wouldn't have to still add homeconnect: to my configuration.yaml file.

It now works for me when I follow the instructions 🤦‍♂, I added the below to my configuration.yaml file and then when I try to add the integration I can follow the link to the Home Connect API site and authorise the integration

homeconnect:
  client_id: YOUR_CLIENT_ID
  client_secret: YOUR_CLIENT_SECRET

Is there a way to remove the requirement for amending the configuration.yaml file and adding this in its entirety through the config flow?

@DavidMStraub
Copy link
Owner

Good question, not sure. I was following the instructions of the new native HA OAuth2 flow:
https://developers.home-assistant.io/docs/en/api_lib_auth.html
As far as I can see this is currently only used by the Somfy component and there the configuration.yaml entry is still mentioned in the docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants