# Point PR janvier 2023

**URL:** <https://community.gladysassistant.com/t/point-pr-janvier-2023/7805>\
**Category:** Développement\
**Created:** [Janvier 2, 2023, 4:49 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805 "2023-01-02T16:49:23Z")\
**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 2, 2023, 4:49 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/1 "2023-01-02T16:49:23Z")

</div>

Salut à tous les [@contributors](https://community.gladysassistant.com/groups/contributors) ! 🙏

Un petit message pour faire un point sur les PRs en cours pour 2023 🙂

Je pense avoir fait le tour des PRs, est-ce qu’il y a des PRs/Issues qui attendent un retour ?

J’essaie de faire au mieux mais Github c’est pas super bien fait côté notifications, les 90% des notifications que je reçois sont inutiles donc je ne m’en sers pas trop ( je reçois une notification à chaque commit dans une PR… super utile, et j’ai jamais réussi à désactiver ça )

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 2, 2023, 5:36 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/2 "2023-01-02T17:36:54Z")

</div>

J’ai corrigé update celle ci

> <https://github.com/GladysAssistant/Gladys/pull/1526>
>
> \### 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)
> \- \[x\] 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\](https://community.gladysassistant.com/) for testing before merging.
> \- \[\] 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
> 
> See the exchanges on Gladys community : https://community.gladysassistant.com/t/plusieurs-erreurs-manques-pour-tasmota-pow-r2/7306
> \- Fixed an error on the intensity feature which was translated into energy.
> \- Fixed the voltage which was /1000 and transformed the voltage from Volt to mVolt for no apparent reason (if on other devices the voltage is retransmitted in milliVolt then the user will have to select the correct unit in the edit menu of the device)
> \- Fixed energy feature category and type to \`ENERGY\_SENSOR\` rather than \`SWITCH\` to get the correct units in the edit devices/features menu
> \- Addition of the following energy data (in particular for the Sonoff POW-Rx model):
> - Apparent power consumption,
> - Reactive power consumption,
> - Total energy,
> - Today's energy.
> \- Fixed the reactive Volt Ampere unit on the front side which in the i18n files was lowercase (var instead of VAr)
> \- Fixed the Watt and kiloWatt units on the front side which were not mentioned in the \`fr.json\` file ("Watt" instead of "Watt (W)" and "KiloWatt" instead of "KiloWatt (kW)") but was good in \`en.json\` file

EDIT: Je ne savais pas que les fichiers de migrations était testés …

Celle là est prête aussi mais pas de feedback user

> <https://github.com/GladysAssistant/Gladys/pull/1581>
>
> Related to =\> https://community.gladysassistant.com/t/vanne-zigbee2mqtt-absence-…dun-parametre/7451
> 
> !\[image\](https://user-images.githubusercontent.com/1528694/190659635-06cee568-24f4-4f74-89ba-50022d059786.png)

---

<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 2, 2023, 5:48 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/3 "2023-01-02T17:48:44Z")

</div>

> [@VonOx](#):
>
> J’ai corrigé update celle ci
> 
> [Add features energy on Tasmota devices, fixes features on server and fixes i18n units energies front by Terdious · Pull Request #1526 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1526)

Les tests ont pas l’air de passer par contre ?

Oui les migrations tournent dans les tests, sinon la DB serait pas là 😃

> [@VonOx](#):
>
> Celle là est prête aussi mais pas de feedback user

Ok, tu peux le relancer sur le forum peut-être ?

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 2, 2023, 6:41 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/4 "2023-01-02T18:41:43Z")

</div>

> [@pierre-gilles](#):
>
> Oui les migrations tournent dans les tests, sinon la DB serait pas là 😃

yes c’est logique, par contre je vois pas comment résoudre le problème:

```auto
20230102171731-update-tasmota-energy-features.js:26:33
  1) "before all" hook: before in "{root}"

  0 passing (6s)
  1 failing

  1) "before all" hook: before in "{root}":
     TypeError: Cannot read properties of null (reading 'id')
      at Object.up (migrations/20230102171731-update-tasmota-energy-features.js:26:33)

```

Je comprends bien l’erreur la table t\_service de la db de test est vide, j’ai créé un seeder mais ça change rien.

Du coup la bonne méthode c’est quoi ? on catch et on stop le fichier de migration ?

---

<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 2, 2023, 6:52 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/5 "2023-01-02T18:52:59Z")

</div>

> [@VonOx](#):
>
> Du coup la bonne méthode c’est quoi ? on catch et on stop le fichier de migration ?

Pourquoi pas, après il faut quand même tester cette migration, sinon on envoie dans la nature une migration pas testée et c’est critique car si la migration crash, Gladys ne démarre plus.

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 3, 2023, 1:14 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/6 "2023-01-03T13:14:53Z")

</div>

La PR tasmota est ok @pierre-gilles

Je relance pour l’autre

---

<div class="post-metadata">

**Author:** ![Romuald\_Pochet](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/romuald_pochet/32/6441_2.png) [@Romuald\_Pochet](https://community.gladysassistant.com/u/Romuald_Pochet)\
**Post date:** [Janvier 4, 2023, 9:09 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/7 "2023-01-04T21:09:27Z")

</div>

Bonsoir,

Concernant la PR Zwave, j’ai déjà eu un retour de Mathieu qui a déjà émis un soucis que j’ai corrigé.  
Etant donné que je dois être le quasi-seul utilisateur de ZWave (42 noeuds ZWave et 72 devices Gladys), le seul point a réellement tester en situation réelle, c’est la mise en place des containers docker, même si on n’a pas de device ZWave. Les 42 noeuds sont fonctionnelles (ce qui n’est pas le cas de tous mes devices Zigbee)

[Zwavejs2mqtt by rpochet · Pull Request #1619 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1619)

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 4, 2023, 10:08 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/8 "2023-01-04T22:08:17Z")

</div>

On a la possibilité de se connecter à un conteneur existant ?

---

<div class="post-metadata">

**Author:** ![Romuald\_Pochet](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/romuald_pochet/32/6441_2.png) [@Romuald\_Pochet](https://community.gladysassistant.com/u/Romuald_Pochet)\
**Post date:** [Janvier 4, 2023, 10:59 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/9 "2023-01-04T22:59:01Z")

</div>

Oui, en choisissant l’option ‹&nbsp;ZwaveJS UI externe&nbsp;›

 ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/6/6ace58de28f7b016e5beea09a080a4b3c3db87c6.png)

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 5, 2023, 6:53 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/10 "2023-01-05T06:53:02Z")

</div>

J’ai qu’un module volet roulant, je test ce soir.

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 5, 2023, 4:59 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/11 "2023-01-05T16:59:36Z")

</div>

@pierre-gilles tu peux actualiser la PR d’alex pour refaire un build ?

> <https://github.com/GladysAssistant/Gladys/pull/1674>
>
> From PR https://github.com/GladysAssistant/Gladys/pull/1642
> 
> Created this PR o…nly for CF builds!

Ty

---

<div class="post-metadata">

**Author:** ![spenceur](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/spenceur/32/1961_2.png) [@spenceur](https://community.gladysassistant.com/u/spenceur)\
**Post date:** [Janvier 5, 2023, 6:52 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/12 "2023-01-05T18:52:50Z")

</div>

Ca pour le coup cest unr tres tres bonne chose pour ceux qui on du migrer et repasser sur zwave js ui il y a pas longtemps !

---

<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 15, 2023, 8:31 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/13 "2023-01-15T08:31:22Z")

</div>

Hello à tous!

Je suis bien rentré en Asie vendredi, et je reviens demain pleine balle sur le développement 🙂

J’ai vu qu’il y avait pas mal de PR en attente, je prend tout ça à bras le corps demain.

Si il y a des sujets «&nbsp;chaud&nbsp;» à votre sens, n’hésitez pas à me dire ici et je regarderais.

En gros là je vais partir sur 4 semaines intensives sur Gladys.

**J’ai plusieurs objectifs :**

- Sortir dès demain toutes les améliorations développées depuis Noël (Duplication de scène, etc… Tout ce qui est sur master en gros)
- Avoir une première version de l’intégration Enedis-Linky en prod pour les utilisateurs Gladys Plus actuels, en «&nbsp;alpha privée&nbsp;»
- Review de toutes les PRs en attente
- Sur Gladys Plus : Rétablissement du plan annuel, ajouter des nouveaux moyens de paiements (je pense à Bancocontact pour nos amis belges, dispo sur Stripe mais que je n’ai jamais activé), réfléchir à des offres différentes: pricing différents ? Des plans 2 ans comme fait NordVPN par exemple? Plan incluant un «&nbsp;starter pack&nbsp;» matériel avec notre partenaire Domadoo? à réfléchir.
- Amélioration de la documentation développeur
- Grosse passe UX sur tout Gladys: faire un ou 2 jours de développements dédiés à l’UX pour mettre en place toutes les petites demandes qui sont toute bête mais qu’il faut bien faire un jour : re-ordonner le dashboard, re-ordonner les scènes, etc…
- Voir comment je peux utiliser le chiffre d’affaire Gladys Plus pour avoir plus de croissance sur le projet. Jeu concours? Partenariat? Payer un presta (si le CA le permet) ?

C’est ambitieux mais très motivant !

Durant ces 4 semaines, plus vous serez réactif, plus je pourrais en faire donc n’hésitez pas!

> [@VonOx](#):
>
> @pierre-gilles tu peux actualiser la PR d’alex pour refaire un build ?

C’est fait ! Le build sera prêt dans quelques minutes 👍 Désolé pour l’attente, en fait pour ça le mieux ça serait que @AlexTrovato tu configure Cloudflare Pages sur ton compte (c’est gratuit), comme ça tu as les builds directement et pas besoin d’attendre que je créé un build «&nbsp;officiel&nbsp;»

> [@Romuald\_Pochet](#):
>
> Oui, en choisissant l’option ‹ ZwaveJS UI externe ›

ça c’est génial ! Bien vu 🕶

---

<div class="post-metadata">

**Author:** ![guim31](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/guim31/32/4840_2.png) [@guim31](https://community.gladysassistant.com/u/guim31)\
**Post date:** [Janvier 15, 2023, 9:37 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/14 "2023-01-15T09:37:11Z")

</div>

> [@pierre-gilles](#):
>
> Grosse passe UX sur tout Gladys: faire un ou 2 jours de développements dédiés à l’UX pour mettre en place toutes les petites demandes qui sont toute bête mais qu’il faut bien faire un jour : re-ordonner le dashboard, re-ordonner les scènes, etc…

Il men fallait pas plus pour passer une bonne journée 😊😊🕶

---

<div class="post-metadata">

**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\
**Post date:** [Janvier 15, 2023, 11:12 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/15 "2023-01-15T11:12:45Z")

</div>

Idem ❤  
Merci @pierre-gilles 😊

---

<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 16, 2023, 4:08 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/16 "2023-01-16T04:08:55Z")

</div>

@VonOx une idée pourquoi les «&nbsp;legacy tags&nbsp;» fail sur le dernier build Gladys version ?

> <https://github.com/GladysAssistant/Gladys/actions/runs/3926693292/jobs/6712720389>

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 16, 2023, 6:38 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/17 "2023-01-16T06:38:42Z")

</div>

Pour une raison que j’ignore il n y’a pas de manifest.

Je reproduis en local

```plaintext
❯ docker manifest inspect gladysassistant/gladys:latest | jq -r '.manifests | to_entries[] | select(.value.platform.architecture == "arm" and .value.platform.variant == "v6").value | .digest'
no such manifest: docker.io/gladysassistant/gladys:latest
❯ docker manifest inspect gladysassistant/gladys:latest
no such manifest: docker.io/gladysassistant/gladys:latest
❯ docker manifest inspect gladysassistant/gladys:v4
no such manifest: docker.io/gladysassistant/gladys:v4
❯ docker manifest inspect gladysassistant/gladys
no such manifest: docker.io/gladysassistant/gladys:latest

```

Je continue l’investigation

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Janvier 16, 2023, 4:45 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/18 "2023-01-16T16:45:03Z")

</div>

Je reproduis sur mon repos de test.

Je tente un downgrade de buildx ( v0.9.1 ) c’est la dernière version qui a fait un build ok ( la latest est en v10.0.0 )

EDIT: ça fonctionne, je fais une PR pour débloquer les releases et je vais monitorer les issues sur le dépot docker

---

<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 17, 2023, 2:14 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/19 "2023-01-17T02:14:41Z")

</div>

> [@VonOx](#):
>
> EDIT: ça fonctionne, je fais une PR pour débloquer les releases et je vais monitorer les issues sur le dépot docker

Génial, merci @VonOx !

---

<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 20, 2023, 4:43 UTC](https://community.gladysassistant.com/t/point-pr-janvier-2023/7805/20 "2023-01-20T04:43:39Z")

</div>

J’ai fais pas mal de merge ce matin 🕶 Merci pour vos PRs !

 ![Screenshot 2023-01-20 at 11.42.53](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/a/a6ae03adfb8ffe39c271f4a8d50764b201a1745a.png)
