Skip to content

Fix Gogogate2 'available' attribute#13728

Merged
MartinHjelmare merged 16 commits intohome-assistant:devfrom
dlbroadfoot:gogogate2
Apr 7, 2018
Merged

Fix Gogogate2 'available' attribute#13728
MartinHjelmare merged 16 commits intohome-assistant:devfrom
dlbroadfoot:gogogate2

Conversation

@dlbroadfoot
Copy link
Copy Markdown
Contributor

I tested some of the recent changes in #13467 by @fabaff and fixed an issue with setting the available property

@dlbroadfoot dlbroadfoot mentioned this pull request Apr 6, 2018
8 tasks
@MartinHjelmare
Copy link
Copy Markdown
Member

Please rebase your branch so that there's only the new commits from this PR.

@dlbroadfoot
Copy link
Copy Markdown
Contributor Author

@MartinHjelmare I'm having some issues rebasing this PR - whoever merges this PR - can they just use the Rebase and merge option in Github please? https://stackoverflow.com/a/39707453/5760250

@MartinHjelmare
Copy link
Copy Markdown
Member

I'll squash and merge. But next time please remember to start your branch from a freshly synced dev branch.

@MartinHjelmare MartinHjelmare changed the title Gogogate2 - Made 'available' private to fix bug Fix Gogogate2 'available' attribute Apr 7, 2018
@MartinHjelmare MartinHjelmare merged commit 58f3690 into home-assistant:dev Apr 7, 2018
@fabaff
Copy link
Copy Markdown
Member

fabaff commented Apr 7, 2018

Thanks. Was really a good idea to make those changes in the GitHub editor.

@MartinHjelmare MartinHjelmare added this to the 0.67.1 milestone Apr 15, 2018
balloob pushed a commit that referenced this pull request Apr 18, 2018
* Fixed bug -  unable to set base readaonly property

* PR fixes

* Added line
@balloob balloob mentioned this pull request Apr 18, 2018
@balloob balloob removed this from the 0.67.1 milestone Apr 18, 2018
@balloob balloob mentioned this pull request Apr 27, 2018
@balloob balloob added this to the 0.67.1 milestone Apr 27, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Sep 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants