muckelba / dynamic-lights-homeassistant

💡 Spicetify extension that changes Home Assistant lights based on the album cover color
16 stars 0 forks source link

colors not changing #4

Open HannistHK opened 1 week ago

HannistHK commented 1 week ago

So i managed to make a SSL instance through Nabu Casa and it goes something like "https://ui.nabu.casa". i verified its ssl status and entered the correct webhook in its place but still no trigger. Heres my YAML code too. image

muckelba commented 1 week ago
  1. Do you see traces in the past? (Click on the top right "Traces" button when editing the automation)
  2. Do you see any errors in spotify when changing the title?
  3. Do you see any errors in the spicetify console when changing the title? (spicetify enable-devtools, right click anywhere in spotify and select Show DevTools)
HannistHK commented 1 week ago
  1. Do you see traces in the past? (Click on the top right "Traces" button when editing the automation)
  2. Do you see any errors in spotify when changing the title?
  3. Do you see any errors in the spicetify console when changing the title? (spicetify enable-devtools, right click anywhere in spotify and select Show DevTools)
  1. These are the traces i see Screenshot_20241104-233156.png

As for 2. And 3. I'll send in a bit

HannistHK commented 1 week ago

Heres the follow up:

  1. No error when changing titles
  2. heres the devlog console image
HannistHK commented 1 week ago

Still waiting....

muckelba commented 1 week ago

Go to the dev console again, change to the Network tab, click on the grey circle 🚫 right next to the red icon. Skip to the next song. Do you see any request that is caused by dynamic-lights-homeassistant.js in the Initiator column? If yes, the extension is probably fine and something is not working on the homeassistant side.

HannistHK commented 1 week ago

Getting these errors 1730983764307.jpg

KinteLiX commented 1 week ago

Edit the webhook trigger as YAML and make sure local_only is false. Screenshot_20241109_132039 @muckelba Please add this to the README. You may also add a notice to the README for users to enable "Music mode" (this will make sure the latency is lower) if they are using yeelight lights.

muckelba commented 1 week ago

Yea, i guess you need to set local_only when using nabu casas solution. Please try that @HannistHK

@KinteLiX interesting, i'll add it to the README, ty

KinteLiX commented 1 week ago

Yea, i guess you need to set local_only when using nabu casas solution. Please try that @HannistHK

@KinteLiX interesting, i'll add it to the README, ty

I'm using cloudflared and had to do this as well. You're welcome!

HannistHK commented 1 week ago

Hey it works now!!!

Thanks @muckelba and @KinteLiX for helping me out, means a lot!

HannistHK commented 1 week ago

Sorry for repeatedly making new comments but there should be a way to saturate the colors a bit mine look a bit bland because of each led not being individually addressable.