@Lokkye Super boulot
Tu as moyen de déplacer le bouton « Désactiver » et de le mettre en rouge ? C’est pas très visuel je trouve de mettre un bouton dans un bandeau d’annonce.
Je mettrais bien le bouton en bas de la page:
@Lokkye Super boulot
Tu as moyen de déplacer le bouton « Désactiver » et de le mettre en rouge ? C’est pas très visuel je trouve de mettre un bouton dans un bandeau d’annonce.
Je mettrais bien le bouton en bas de la page:
Je confirme, parfaitement fonctionnel.
Bouton déplacé comme tu as dit (la suggestion de @pierre-gilles est excellente) et container supprimé à la désactivation.
Plus qu’à écrire la doc qui renvoie actuellement vers un 404
Bravo ! Je n’aurai bientôt plus d’excuse pour me lancer dans node-red (pour contrôler l’état de la voiture…) !
Voici le résultat avec le bouton déplacé
J’ai refait une image docker avec les changements
Merci pour les changements @Lokkye !
C’est mieux mais j’ai quelques retours/questions :
Voilà c’est tout pour moi, des petits détails d’UX mais qui à mon sens ont toute leur importance
Super cette fonctionnalité.
Je suis intéressé mais n’ai pas l’intégration dans Gladys.
Y a t il une procédure « standard » pour installer cette intégration?
Merci.
Pour le moment c’est développement, vu l’avancée je pense que ça ne devrait pas tarder à arriver dans une prochaine version
Si tu es pressé tu peux aussi installer Node-Red toi même à côté de Gladys, c’est ce que beaucoup ont fait ! (avec ce tuto en prime : Nouvelle vidéo: Intégrer Node-RED avec Gladys Assistant en MQTT!)
C’est caché maintenant
Oui
C’est fait
Elle est basé sur « config.localApiUrl » plus le port qui est retourné par l’api
C’est fait
Tu as complètement raison
Comme dab, j’ai sortie une nouvelle image docker
Merci pour tous les changements
J’ai essayé de faire un test de bout en bout mais j’ai eu des petits soucis.
Au début, j’ai essayé de lancer le container mais j’avais plus de disque disponible sur ma VM de test (là c’est pas lié à la PR, au moins c’est l’occasion de tester le message d’erreur)
(D’ailleurs petite typo sur « serveur »)
J’ai trouvé le comportement du « Désactiver » un peu bizarre, ça fait un saut de gauche à droite:
Ensuite, je vide mon disque et Node-RED se lance bien:
(Petite remarque design sur le screenshot)
Sauf que si j’essaie de me connecter à Node-RED, j’ai une erreur:
En regardant les logs:
2023-09-08T10:36:31+0200 <info> checkForContainerUpdates.js:13 (NodeRedManager.checkForContainerUpdates) NodeRed: Checking for current installed versions and required updates...
2023-09-08T10:36:31+0200 <info> checkForContainerUpdates.js:17 (NodeRedManager.checkForContainerUpdates) NodeRed: update #2 of the container required...
2023-09-08T10:36:31+0200 <info> checkForContainerUpdates.js:34 (NodeRedManager.checkForContainerUpdates) NodeRed: update #2 of the container done
2023-09-08T10:36:31+0200 <info> installContainer.js:44 (NodeRedManager.installContainer) Nodered: is being installed as Docker container...
2023-09-08T10:36:31+0200 <info> installContainer.js:45 (NodeRedManager.installContainer) Pulling nodered/node-red:latest image...
2023-09-08T10:36:56+0200 <info> installContainer.js:49 (NodeRedManager.installContainer) Nodered: Preparing environment...
2023-09-08T10:36:56+0200 <info> configureContainer.js:16 (NodeRedManager.configureContainer) NodeRed: Docker container is being configured...
2023-09-08T10:36:56+0200 <info> configureContainer.js:34 (NodeRedManager.configureContainer) NodeRed: configuration file already exists.
2023-09-08T10:36:56+0200 <info> installContainer.js:52 (NodeRedManager.installContainer) Creation of container...
2023-09-08T10:36:56+0200 <info> installContainer.js:57 (NodeRedManager.installContainer) NodeRed: successfully installed and configured as Docker container
2023-09-08T10:36:56+0200 <info> configureContainer.js:16 (NodeRedManager.configureContainer) NodeRed: Docker container is being configured...
2023-09-08T10:36:56+0200 <info> configureContainer.js:34 (NodeRedManager.configureContainer) NodeRed: configuration file already exists.
2023-09-08T10:36:56+0200 <info> installContainer.js:80 (NodeRedManager.installContainer) NodeRed: container is (re)starting...
2023-09-08T10:37:01+0200 <info> installContainer.js:86 (NodeRedManager.installContainer) NodeRed: container successfully started
Je suppose que du coup le fichier de configuration est encore l’ancien!
Du coup, si jamais j’ai déjà configuré Node-RED une fois, je suis bloqué (je pense que l’ancien mot de passe est encore dans le fichier). En testant le hash contre le mot de passe, effectivement c’est pas le même mot de passe. Est-ce que tu supprime bien le dossier /var/lib/gladysassistant/node-red si l’utilisateur décide de trasher l’intégration Node-RED ?
Au niveau des traductions, j’ai quelques feedbacks:
Autre question plus générale, est-ce qu’on figerait pas la version de Node-RED ? (Au moins une version majeure ou mineure)
J’ai peur qu’on se retrouve avec les mêmes bugs qu’on a eu sur Mosquitto où au beau milieu de l’été une version pourrie a été poussée de leur côté et ça a tout cassé chez les utilisateurs Gladys.
Je sais qu’on fait pas ça sur Zigbee2mqtt, mais je sais que Node-RED ils sont assez « agressifs » sur les mises à jour et souvent de ce que je lis sur le forum, les mises à jour cassent tout.
Par exemple, on pourrait utiliser le tag:
nodered/node-red:3.1
Pour les mises à jour, on pourra les faire manuellement côté code, en faisant des tests avant de changer.
Tu en penses quoi ?
Merci en tout cas pour l’intégration et tout le travail, c’est du très beau boulot
@Lokkye
Chapeau bas pour ce développement ! Je pense vraiment qu’il était une nécessité pour l’utilisateur lambda car il ne nécessitera plus de faire des manips en ligne de commande pour avoir node-red avec Gladys et permettra à tout à chacun d’intégrer à Gladys n’importe quel matériel via node-red comme je l’ai fait avec mon IPX800V5 ou de couvrir n’importe quel besoin ! Par exemple ici, j’envoie à Gladys l’a hauteur et l’azimut du soleil à intervalle fixe pour piloter des volets roulants !
Bon maintenant l’intégration de la reco vocale (Docker) et le Text To Speech (Docker images - TTS 0.16.6 documentation) dans Gladys via des container ?
Merci encore pour ce dev et vivement qu’il soit en prod !
Merci @pierre-gilles pour le test complet
A mince, j’avais oublié. Je viens de commit le changement
Je viens de tous fixer dans le dernier commit.
Tu as raison. je prefere moi aussi fixé une version, au moins on sait ce qui est vraiment installé.
C’est corrigé dans le dernier commit
L’image docker est en cours de création avec tous les chagements
Merci pour les changements @Lokkye, j’ai stoppé puis relancé Node-RED et maintenant ça marche
Quelques retours:
Le lien de la documentation a une petite typo « noder-red » au lieu de « node-red »
Toujours le petit « saut » lors de la désactivation du service (même vidéo que dans mon précédent message)
Sinon tout le reste est bon Beau boulot !
Oops, j’avais oublié, c’est fixé maintenant
Fixé
Qu’est ce que tu penses de cette version ?
Merci pour tous les changements!
C’est beaucoup mieux
Si tout est bon pour toi tu peux relancer un build et je ferais une review finale ?
Comment ça se passe quand on fixe une version coté Gladys avec watchtower ? Watchtower ne mets pas à jour le conteneur en dernière version ?
Top je vais tester @Lokkye !
Non ! Watchtower met à jour un container si l’image référencée est mise à jour, pour passer d’une version à l’autre, il faut utiliser un tag « large » dont l’image est écrasée à chaque version.
Par exemple, Node-RED met à disposition un tag « latest », « 3.1 », et « 3.1.1 »:
Dans le cas où on fixe la version, c’est à nous côté Gladys de tester chaque nouvelle version et d’ensuite la proposer aux utilisateurs après vérification que ça n’explose pas tout
@Lokkye sûr que le build est fini ? j’ai pull, relancé le container, et malgré ça j’ai toujours l’ancienne version (pas de margin, et le bouton qui saute toujours)
Question idiote peut-être mais… Tu es sûr du refresh de ton navigateur ?
Je me permet de te demander car j’ai eu le cas la dernière fois, malgré un rmi/pull complet…
Yes yes j’ai rafraichi, je pense que l’image est pas à jour ou a été buildé sur une ancienne version?