Sécurité enfant très bien
2 « J'aime »
On est d’accord du coup sur le faite que que je peut supprimer le device Mode de contrôle d'accès
? Car je voit pas l’utiliter de ce bouton 3 états sans avoir d’info sur les actions.
Hizo
Novembre 28, 2023, 8:02
17
Oui, il n’est pas adapté.
@Hizo ,
Voila une image docker si tu veux essayé
willde71/gladys-test:zigbee2mqtt-childlock
@pierre-gilles ,
Voila la PR
GladysAssistant:master
← William-De71:zigbee2mqtt-child-lock
opened 10:07PM - 28 Nov 23 UTC
### 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:
- [ ] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front/server)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] 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)
- [ ] 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
https://github.com/GladysAssistant/Gladys/issues/1886
Replace access control mode by child lock binary feature in smart plug device.
Old feature:
![image](https://github.com/GladysAssistant/Gladys/assets/11477113/274a6ae1-2c5a-4faa-8d9d-94d82d7d76bc)
New feature:
![image](https://github.com/GladysAssistant/Gladys/assets/11477113/899d952f-68f5-478c-af23-fee3eca66509)
Merci ! J’attend un retour de testeur en réel et ensuite c’est bon pour moi !
1 « J'aime »
De mon côté j’ai testé en réel mais si quelqu’un pouvait le faire aussi c’est bien
Bon, il y a pas l’air d’avoir trop d’engouement, si tu as testé en réel de ton côté je te fais confiance
Est-ce que tu peux mettre à jour ta PR (il y a des conflits) et je mergerais ?
Ok je m’en occupe ce soir ou dans le week end.
@Hizo , peux-tu faire un test comme tu avais demandé la modification?
Hizo
Décembre 8, 2023, 2:33
23
Je sais plus trop comment on fait,
il faut juste ajouter willde71/gladys-test:zigbee2mqtt-childlock
?
ou il faut arrêter gladys ?
Parfois, un développeur veut proposer une image Docker « de test » sur le forum pour permettre à des utilisateurs de tester une nouvelle intégration avant qu’elle soit déployée en production.
Contrairement à l’image Docker « officielle » qui est hébergée sur le compte du projet gladysassistant/gladys, une image de test sera hébergée sur le compte Docker du développeur proposant l’image. Exemple: jeanfrancois/gladys.
Pour lancer une image Docker de test, c’est le même process que décrit sur la …
Tu as les explication ici.
Tu stoppes,
Tu supprimes,
Tu vas chercher la nouvelle image.
Disclaimer : Je précise quand même que ce tutoriel s’adresse à un public avancé
@Hizo si tu ne te sens pas de le faire, ne le fais pas. Vu la taille du changement, ça ne vaut pas le coup de casser son installation pour ça
Je suis d’accord si tu n’est pas a l’aise @Hizo ne test pas
Il y a @bab85 qui a voulu tester une image et a cassé sa prod gladys.
Sachant que pour moi la PR est bonne, donc ça partira en prod de toute façon, et @_Will_71 a déjà testé
Hizo
Décembre 8, 2023, 2:50
28
Ouais, je suis pas trop chaud, j’arrive enfin à tout faire fonctionner comme je veux
Merci, au pire, je testerai en prod.
1 « J'aime »
_Will_71:
C’est fait.
C’est mergé et ça partira dans la prochaine version de Gladys ! Merci pour la PR @_Will_71
2 « J'aime »