-
Notifications
You must be signed in to change notification settings - Fork 46
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
Integration removes lovelace resources at startup. #287
Comments
I did see this a couple of times when developing the schedule card and reworked the way it adds and added lovelace as a dependancy to the manifest so that it had loaded before it tries to add. I would say, however, that if these are already added to resources, it doesn't try to add them at each restart. Can you confirm that it really is happening at each restart or just when you have deleted them from resources and restarted. Can you also provide a diagnostics download from the wiser integration so i can see your install method and what else you have loading tomsee if it is related to that. |
This happens every time Home Assistant is restarted. To test, I commented out lines 381 to 389 of the |
Ok thx. I think this can only happen if it tries to add the resources before lovelace has fully loaded. Can i assume you are running this on good hardware and it restarts/loads HA pretty quickly? I will look to find a more robust way of ensuring lovelace has loaded its resources before it tries to read/add the wiser ones. Will let you know when i have something if you would be able to test it for me. |
Home Assistant is launched in a LXD container with 2 cores of an i7-10700K and 12GB of Ram (with an NVMe SSD + a RAM cache). The reboot only takes a few seconds (usually between 2 and 5 sec). So yes a pretty good hardware for HA. No problem, I can do additional tests if needed. |
@FloX-Dev - I have amended the code in the Dev branch to wait for Lovelace resources to have loaded before adding/checking the wiser ones. Would you be able to test for me? You will need to delete the wiser resources and restart to know if working properly. Thanks |
The resources are no longer deleted, but the two JS files for the integration are not added. Here is an error in the logs:
|
Sorry, finding hard to test as even though I have pushed my dev env upto 6 cores of an Ryzen 3900X and 16GB mem, cannot recreate your issue. Pull from dev again and try now. Thx |
It works! I have tried a dozen times, the resources are correctly added without removing the others. |
Ah, yes that maybe the reason. Anyway, great news, will include this in next 3.1.4 release. |
Have released fix in 3.1.4. Regards |
Hello,
I have a problem with lovelace resources with the Wiser integration. When the integration is activated, the resources other than those of wiser seem to be deleted at each restart! This happens since I switched to the release (R3.1.2), I skipped some version, I can't say exactly from which version the problem started to occur.
The problem seems to be when injecting "/wiser/wiser-schedule-card.js" and "/wiser/wiser-zigbee-card.js" to lovelace resources.
Before HomeAssistant restart:
After restart:
The text was updated successfully, but these errors were encountered: