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

Light: Add effects list #142

Open
mlc76520 opened this issue Feb 26, 2023 · 5 comments
Open

Light: Add effects list #142

mlc76520 opened this issue Feb 26, 2023 · 5 comments

Comments

@mlc76520
Copy link

mlc76520 commented Feb 26, 2023

First thanks for this and the hard work. This library is really amazing.
In the Light Module, I'd like to have the opportunity to get a list of effects in home assistant mapped to some custom effects available on the arduino side. ie a rainbow effect.

Hope you will consider the request if not too challenging,

Best regards,

@mlc76520 mlc76520 changed the title Add effects list Light: Add effects list Feb 26, 2023
@addyvandendoel
Copy link

Add me to this please!
I also Like to be able to set effects for the light. Used the library for connecting a color ledstrip WS2813 to and esp8266. Getting it lit up with this is amazing, just missing out the effects option now.
I guess I could fix this myself with the onMessage( ) function, but would be really nice to have an effect list feature for the light.

@martaisty
Copy link
Contributor

Plus 1 on this. 👍

@martaisty
Copy link
Contributor

@mlc76520 , @addyvandendoel Hello 👋 It's been a while since the issue was created 😄 I also needed this feature and implemented it in PR #178. Until it's merged, feel free to use changes on my fork https://github.com/martaisty/arduino-home-assistant/tree/feature/light-effects-list

@martaisty
Copy link
Contributor

Hi @dawidchyrzynski 👋 I see that you are actively doing changes on the project. I was wondering if there is a chance to get this issue resolved. I will resolve the conflicts on my PR and kindly ask to look at it 😃

@dawidchyrzynski
Copy link
Owner

Hi @martaisty, no problem. Let's aim to merge it asap.

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

No branches or pull requests

4 participants