# Evolution du service Z-Wave JS UI

**URL:** <https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692>\
**Category:** Développement\
**Created:** [Janvier 26, 2024, 8:43 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692 "2024-01-26T08:43:14Z")\
**Posts on this page:** 20\
**Page:** 1

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Janvier 26, 2024, 8:43 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/1 "2024-01-26T08:43:14Z")

</div>

Salut @Sescandell 🙂

Je créé ce sujet pour discuter de l’évolution du service Z-Wave JS UI et de ta PR !

Déjà merci de t’être lancé sur le sujet, c’est vraiment chouette ce que tu as fais.

Je t’ai fais une première review:

> <https://github.com/GladysAssistant/Gladys/pull/2002#pullrequestreview-1845254303>
>
> \### Pull Request check-list
> 
> To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items:
> 
> \- \[x\] If your changes affects code, did your write the tests?
> \- \[x\] Are tests passing? (\`npm test\` on both front/server)
> \- \[x\] Is the linter passing? (\`npm run eslint\` on both front/server)
> \- \[x\] Did you run prettier? (\`npm run prettier\` on both front/server)
> \- \[\] ~If you are adding a new features/services, did you run integration comparator? (\`npm run compare-translations\` on front)~
> \- \[x\] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community (\[french forum\](https://community.gladysassistant.com/)/\[english forum\](https://en-community.gladysassistant.com/)) for testing before merging.
> \- \[\] ~If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)~
> \- \[\] ~If you are adding a new features/services which needs explanation, did you modify the user documentation? See \[the GitHub repo\](https://github.com/GladysAssistant/v4-website) and the \[website\](https://gladysassistant.com).~
> \- \[\] Did you add fake requests data for the demo mode (\`front/src/config/demo.js\`) so that the demo website is working without a backend? (if needed) See \[https://demo.gladysassistant.com\](https://demo.gladysassistant.com).
> 
> NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
> 
> \### Description of change
> 
> Binary Switch support tested with real life device Fibaro Wall plug - FGWPE-102.
> 
> Manage displaying state and updating state from the UI. Also working with scenes (when the current implemented door sensor opens, it turns on the Binary Switch)
> 
> !\[image\](https://github.com/GladysAssistant/Gladys/assets/1559970/12ad0c0a-b3fb-411d-9b76-e890e4e040dc)
> 
> !\[image\](https://github.com/GladysAssistant/Gladys/assets/1559970/1952aa7d-40fa-4391-ba80-e5d84549457e)
> 
> !\[image\](https://github.com/GladysAssistant/Gladys/assets/1559970/1b2885e8-475c-4c8e-88db-14196a8a1081)

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Janvier 26, 2024, 12:04 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/2 "2024-01-26T12:04:57Z")

</div>

De mon côté j’ai testé en réel la PR et cela détecte des devices que j’ai et je peux les contrôler dans Gladys

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 2, 2024, 10:37 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/3 "2024-02-02T10:37:31Z")

</div>

@Sescandell Je viens de voir que tu avais mis à jour ta PR, merci pour les correctifs ! 🙂

Pense à me mettre un petit message ici la prochaine fois, sinon je ne sais pas qu’il y a eu des changements sur Github !

J’ai relancé les tests !

@Will_71 Tu peux re-tester après les derniers changements de @Sescandell ? 😉 il faudra que tu re-créé les appareils Z-Wave par contre

---

<div class="post-metadata">

**Author:** ![Sescandell](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/sescandell/32/3759_2.png) [@Sescandell](https://community.gladysassistant.com/u/Sescandell)\
**Post date:** [Février 2, 2024, 11:29 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/4 "2024-02-02T11:29:57Z")

</div>

Je ne voulais pas être envahissant entre les notif github les notif ici tout ça tout ça.

Je m’en souviendrai 👍

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Février 2, 2024, 11:32 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/5 "2024-02-02T11:32:52Z")

</div>

Oui pas de souci je re-testerai dans le week-end

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 2, 2024, 1:22 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/6 "2024-02-02T13:22:38Z")

</div>

@Sescandell au contraire hésite pas 🙂

Sur GitHub c’est pas pratique, chaque commit fait une notification donc la partie notification de mon compte est inutilisable (j’ai des milliers de notifications dedans 😅)

Donc je préfère le forum pour prévenir que c’est bon ! Par contre une fois que tu me préviens je vais voir les messages GitHub sans problèmes

---

<div class="post-metadata">

**Author:** ![Sescandell](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/sescandell/32/3759_2.png) [@Sescandell](https://community.gladysassistant.com/u/Sescandell)\
**Post date:** [Février 2, 2024, 5:26 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/7 "2024-02-02T17:26:35Z")

</div>

J’ai revert le fichier package-lock.json du service usb. Dis-moi.

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Février 4, 2024, 10:08 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/8 "2024-02-04T22:08:44Z")

</div>

> [@pierre-gilles](#):
>
> @Will_71 Tu peux re-tester

J’ai re-testé mail il y a du changement chez moi.  
Les devices sont toujours reconnus mais ceux qui était gérés par Gladys ne le sont plus.

J’essaierais de regarder pourquoi quand j’aurais 5 min.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 5, 2024, 8:47 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/9 "2024-02-05T08:47:02Z")

</div>

> [@Will\_71](#):
>
> J’ai re-testé mail il y a du changement chez moi.  
> Les devices sont toujours reconnus mais ceux qui était gérés par Gladys ne le sont plus.

C’est normal, la première PR de @Sescandell modifiait le external\_id, je lui ai demandé dans la PR de revert ce changement et de revenir au external\_id actuellement en production.

Donc si tu avais déjà créé les appareils Zwave dans ton premier test, il faut les supprimer puis les re-créer ( j’ai demandé le changement justement pour que le souci que tu as eu ne s’applique pas aux utilisateurs en production 😃 pas de breaking change )

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Février 5, 2024, 8:51 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/10 "2024-02-05T08:51:36Z")

</div>

> [@pierre-gilles](#):
>
> Donc si tu avais déjà créé les appareils Zwave dans ton premier test, il faut les supprimer puis les re-créer

Je ne suis pas reparti de la même installation donc mes capteurs n’étaient pas créés dans Gladys.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 5, 2024, 10:03 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/11 "2024-02-05T10:03:54Z")

</div>

Ah ok, effectivement dans ce cas là il y a un souci 🙂

---

<div class="post-metadata">

**Author:** ![Sescandell](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/sescandell/32/3759_2.png) [@Sescandell](https://community.gladysassistant.com/u/Sescandell)\
**Post date:** [Février 8, 2024, 3:50 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/12 "2024-02-08T15:50:55Z")

</div>

Hello

Possible d’avoir plus d’infos svp : type de devices , logs , qu’est-ce qui ne fonctionne pas ? etc.

Merci,

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Février 8, 2024, 4:12 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/13 "2024-02-08T16:12:49Z")

</div>

J’essaie de te donner plus d’info plus tard en ce moment je fais beaucoup de déplacement pour le boulot alors j’ai pas trop le temps.  
Tout ce que je peux te dire pour l’instant c’est que sur ta première image j’ai des devices qui était reconnu (enfin une partie seulement) switch ON/OFF. J’avais bien la remontée dans Gladys et je pouvais piloter depuis Gladys.

Avec la dernière modification cela ne fonctionne plus, les devices sont affichés mais avec le message qu’ils ne sont pas encore gérés par Gladys.

Aucun logs spécifiques.

Les devices sont de marque qubino.  
Je pourrais me connecter à zwavejs ui ce soir pour te donner plus de details des devices.

---

<div class="post-metadata">

**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\
**Post date:** [Février 8, 2024, 11:29 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/14 "2024-02-08T23:29:34Z")

</div>

Bon j’ai fait une erreur quand j’ai cloner ton repo car je n’ai plus la gestion du binary switch dans le code  
Donc forcement cela ne fonctionne pas.

Donc je confirme que maintenant c’est bon mes devices (binary switch) fonctionnent!

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 9, 2024, 10:07 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/15 "2024-02-09T10:07:35Z")

</div>

Je viens de faire la review de mon côté, et j’ai testé en réel avec mon capteur Fibaro d’ouverture de porte : ça marche nickel 👌 ([Review](https://github.com/GladysAssistant/Gladys/pull/2002#pullrequestreview-1872039368))

Beau boulot 👏👏

J’ai approve la PR, est-ce que tu peux juste me faire une PR sur la documentation pour adapter le texte et dire qu’on supporte les prises connectées ? 🙂

Ensuite je mergerais sur master et ça partira dans la prochaine version de Gladys !

---

<div class="post-metadata">

**Author:** ![Sescandell](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/sescandell/32/3759_2.png) [@Sescandell](https://community.gladysassistant.com/u/Sescandell)\
**Post date:** [Février 9, 2024, 6:20 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/16 "2024-02-09T18:20:05Z")

</div>

Je ne suis pas certain de la procédure pour la doc’.

J’ai fait quelque chose, n’hésite pas à me dire si je me suis raté.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 9, 2024, 6:37 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/17 "2024-02-09T18:37:04Z")

</div>

C’est un repo Docusaurus (un outil de documentation basé sur Node.js)

Tu as les instructions dans le README pour mettre en place un environnement !

N’hésite pas si tu veux un coup de main 🙂

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 12, 2024, 1:32 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/18 "2024-02-12T13:32:41Z")

</div>

@Sescandell Merci pour la PR côté doc 🙂

J’ai mergé la PR côté Gladys, ça partira dans la prochaine version de Gladys ! (Probablement semaine prochaine)

---

<div class="post-metadata">

**Author:** ![Sescandell](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/sescandell/32/3759_2.png) [@Sescandell](https://community.gladysassistant.com/u/Sescandell)\
**Post date:** [Février 12, 2024, 3:50 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/19 "2024-02-12T15:50:59Z")

</div>

C’est parti pour une nouvelle PR alors 🙂

J’ai vu que le door/windows sensor a un capteur de température intégré. J’attaque par ça !

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Février 12, 2024, 3:54 UTC](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692/20 "2024-02-12T15:54:08Z")

</div>

Excellent, avec plaisir pour une nouvelle PR !

N’hésite pas si tu as besoin d’aide, ou de matériel 🙂

[Page suivante](https://community.gladysassistant.com/t/evolution-du-service-z-wave-js-ui/8692.md?page=2)
