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

'Yeelight Wizard' do not saves the configuration #133

Closed
fallenruby opened this issue Oct 24, 2021 · 8 comments
Closed

'Yeelight Wizard' do not saves the configuration #133

fallenruby opened this issue Oct 24, 2021 · 8 comments

Comments

@fallenruby
Copy link

Hi,
I'm running v17.0.0.0. When I try to use the wizard for the yeelight (LED Hardware -> LED Controller -> Controller type: Network - yeelight), the button 'Save' does not work, therefore I can't add the bulb. Works perfectly manually: but I was curious about the auto-configuration in this case, and specifically if the wizard 'fine tunes' in some way the LED layout section.

image

@awawa-dev
Copy link
Owner

Could you open developer tools in your browser ( https://balsamiq.com/support/faqs/browserconsole/ ), switch to it's console tab and monitor what's happening there when you press 'Save' button in HyperHDR. Paste the error here if occurs.

@fallenruby
Copy link
Author

wizard.js:1445 Uncaught TypeError: Cannot set properties of undefined (setting 'enable') at HTMLButtonElement.<anonymous> (wizard.js:1445) at HTMLButtonElement.dispatch (jquery.min.js:2) at HTMLButtonElement.v.handle (jquery.min.js:2) (anonymous) @ wizard.js:1445 dispatch @ jquery.min.js:2 v.handle @ jquery.min.js:2

@awawa-dev
Copy link
Owner

awawa-dev commented Oct 24, 2021

I suspect you have just created new instance and then you try to add Yeelight lamp. But the config probably is not ready yet. Before we can go further with a fix, please restart HyperHDR and run the wizard again. It should work. Do not create new instance but use that you already have. Again note the error if it still doesn't work.

@fallenruby
Copy link
Author

Correct, I've created a new instance and added the lamp.
That was it, after the restart it saves correctly, thank you.

@awawa-dev
Copy link
Owner

Great! I'll include conditional check to fix it anyway.

@awawa-dev
Copy link
Owner

Patch has been committed to the current development version.

@awawa-dev
Copy link
Owner

@fallenruby
One more thing: I see that you have unsupported Yeelight on the screenshot. I guess it's a ceiling lamp.
Currently I'm implementing new support for these lamps, it's half done for Yeelight 650 (I'm not sure the final result yet because the stream fast communication is not supported).
May I know what is your lamp model and does it have a color ring or it's only white lamp?

@fallenruby
Copy link
Author

Hey @awawa-dev that's actually the yeelight monitor lamp, product code YLTD001 (it does have the main light and some color LEDs on the back as ambient lighting).

But thank you for your interest!

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

No branches or pull requests

2 participants