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.

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

Add my service for light and cover devices #236

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

iMicknl
Copy link
Owner

@iMicknl iMicknl commented Aug 22, 2020

Closes #220

For now, I did register tahoma.cover_my and tahoma.light_my since I was not able to link tahoma.my to multiple entities...

I tried creating tahoma.cover.my and tahoma.light.my as well, however in this case the services.yaml doesn't get read correctly.

@github-actions github-actions bot added the enhancement New feature or request label Aug 22, 2020
@tetienne
Copy link
Collaborator

Yes, sadly it seems we haven't the choice to repeat the code.

@iMicknl iMicknl merged commit ee58468 into master Aug 24, 2020
@iMicknl iMicknl deleted the enhancement/add_my_service branch August 24, 2020 09:04
iMicknl added a commit that referenced this pull request Aug 27, 2020
commit 7af2d98
Author: Mick Vleeshouwer <Mick.Vleeshouwer@microsoft.com>
Date:   Thu Aug 27 15:06:33 2020 +0200

    Add custom session, handle HTTP_CONNECT_ERRORS and retry on ServerDisconnectedError (#237)

commit ee58468
Author: Mick Vleeshouwer <mick@imick.nl>
Date:   Mon Aug 24 11:03:59 2020 +0200

    Add `my` service for light and cover devices (#236)
iMicknl added a commit that referenced this pull request Aug 27, 2020
commit 7af2d98
Author: Mick Vleeshouwer <Mick.Vleeshouwer@microsoft.com>
Date:   Thu Aug 27 15:06:33 2020 +0200

    Add custom session, handle HTTP_CONNECT_ERRORS and retry on ServerDisconnectedError (#237)

commit ee58468
Author: Mick Vleeshouwer <mick@imick.nl>
Date:   Mon Aug 24 11:03:59 2020 +0200

    Add `my` service for light and cover devices (#236)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose my position for covers via a service
3 participants