Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix deCONZ retry mechanism for setup #21492

Merged
merged 1 commit into from Feb 27, 2019
Merged

Fix deCONZ retry mechanism for setup #21492

merged 1 commit into from Feb 27, 2019

Conversation

Kane610
Copy link
Member

@Kane610 Kane610 commented Feb 27, 2019 •
edited

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

@Kane610
Since moving retry mechanism to exception based we can't store gatewa…
5ebd114 
…y to hass data until after setup is complete
@ghost ghost assigned Kane610 Feb 27, 2019
@ghost ghost added the in progress label Feb 27, 2019
@skeptizzle
Copy link

I really appreciate your time and effort on this @Kane610 !

@Kane610 Kane610 merged commit 2482816 into home-assistant:dev Feb 27, 2019
@ghost ghost removed the in progress label Feb 27, 2019
@Kane610 Kane610 added the bug label Feb 28, 2019
@Kane610 Kane610 added this to the 0.89.0 milestone Feb 28, 2019
@Kane610 Kane610 deleted the deconz-fix-retry-bug branch March 24, 2019 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Kane610 Kane610

Labels
bug cherry-picked cla-signed integration: deconz small-pr PRs with less than 30 lines.
Projects
None yet
Milestone
0.89.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Kane610 @skeptizzle @balloob @homeassistant