-
Notifications
You must be signed in to change notification settings - Fork 12
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
additional check in case of null eqLogic #1598
base: develop
Are you sure you want to change the base?
Conversation
@TesteursFous : Pour tester cette PR, il faut supprimer manuellement le répertoire d'un plugin de votre choix. Il faut ensuite se rendre sur la page Diagnostics/Equipement et s'assurer que cette page n'est pas vide ou blanche |
J'avoue que je ne comprends pas bien le sens du fix de cette PR ... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quand tu as un plugin qui plante tout ND c'est le derniers recours "simple". |
Le soucis, c'est que cette solution ne fonctionne que sur 1 page ou 2, mais si ton nextdom est planté, ça cassera sur les autres pages. |
Le plugin incriminé est toujours présent sur mon installation. C'est la seule page qui fait ça et uniquement sur nextdom (c'est ok sur jeedom) |
@fensoft j'obtiens ça à la place d'une page vide Mais je n'ai pas ce que l'on devrait avoir. |
C'est pas vraiment mieux. Un vrai fix tel que celui que j'ai mis est nécessaire |
J'ai mis ton code en fait |
J'imagine que le code a changé depuis la PR |
Le soucis de cette correction, c'est qu'elle fix 1 affichage mais pas le reste des autres problèmes. Donc au pire le dashboard s'affichera, mais tout pourra crasher plus loin |
cf #1577