-
-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reconnect on Failed Authentication #39
Conversation
See HandyHat#14 for discussion of issue
Not quite sure what is happening, so also log URL
"documentation": "https://github.com/ColinRobbins/ha-hildebrandglow-dcc", | ||
"issue_tracker": "https://github.com/ColinRobbins/ha-hildebrandglow-dcc/issues", | ||
"codeowners": ["@handyhat","@ColinRobbins"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines are problematic given a PR into upstream.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well spotted, this is now removed from the PR.
(I need it in my main branch as some people are using my fork, as HandyHat does not seem to be accepting PR's at the moment. Thus people using my fork, need to amended version to HACS updates to work.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is pretty confused, looking again, as it mixes in a new feature too.
Now sure it’s worth fixing, as it’s been superseded by #40
Description
Fix for issue #14
await
to call of failed_auth_handler, otherwise its not calledMotivation and Context
Fix fo #14
How Has This Been Tested?
Locally in test and live environments
Types of changes