-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Replace flask branch with beta and fix it's dir structure #24
Conversation
Number of 'XX' in the placeholder was incorrect. It should be 6 instead of 5.
I checked this version and all working fine. |
@HepoH3 I tried out your branch, but I cannot seem to generate the |
See: #21 (comment). I can't tell why config.json is not creating while install, I used this integration only to simulate hue bridge and use it outside of home assistant or any other hubs like deconz and control lights via hue sync on PC for the music lights (which still works pretty bad but I think it's either yeelight bulbs or hue sync software problems). The goal of this PR is to fix install issues of diyhue-beta branch that related to the changes in dir structure of the repository. The config.json problem seems to be related to some other changes that I don't know how to fix. |
The only thing I can think of is the addition of the |
If that the case, I'll revert my commit that return host_network back, but I faced problems with yeelight bulbs discovering without that parameter. |
If that is actually the issue, then I'd say it's a bug. But that should then be relayed to the diyhue devs. But let's first see if that's the actual issue or not 😛. |
Okay, so after a shitload of trial and error I've come to the realization that it is something on the diyhue teams side. Even on a fresh install inside of Docker. It won't create a config file. However, it does create one when using the master branch. Which brings me to the assumption where I believe this is a bug on their end. |
Hey looks like the config path moved: diyhue/diyHue#646 (comment) I symlinked |
hey guys, i will fix a lot of bugs here soon. had a bit of family trouble in the last months but now everything is fine again. |
Also I've reverted
host_network:true
because of #18, and fixed mac placeholders.Note, that in the repository.json I've add 'fork' suffix to distinguish repositories on the addons page.
#21, #15