VonOx
Septembre 4, 2021, 9:03
1
Bonjour à vous,
après quelques temps d’utilisation, voici quelques retours et demandes en vrac, ce ne sont que des retours, aucune obligation ou mécontentement de ma part.
Le travail déjà réalisé est génial !
Scènes
Permettre la modification de l’icône des scènes une fois créées (on peut les renommer mais pas changer l’icône).
Permettre de dupliquer des scènes afin de pas tout recréer de 0 lorsqu’on fait des scènes très proches.
Préciser le matériel ayant déclencher la scène lorsqu’il y a p…
Le titre parle de lui même, le besoin est d’avoir une box qui permette d’afficher les features des devices indépendamment de la pièce. Exemple on pourrait regrouper toutes les devices features batterie pour avoir d’un seul coup d’oeil le niveau de batterie de nos périphériques.
Je penses aussi à @Terdious , la box pourrai avoir un filtre optionnel sur le type de feature ( température / batterie etc … ) car avec une grosse installation la liste va probablement être longue comme le bras.
Le but est d’éviter ça:
ou encore ceci:
J’aime beaucoup !! Et merci pour la petite pensée @VonOx
Hizo
Mars 7, 2022, 11:04
4
Bonjour à tous,
je voulais savoir si la proposition était retenue et vous aviez pu avancer dessus
Merci beaucoup
Bonjour @Hizo Personne n’a commencé ce développement à ma connaissance.
Pour ma part, je fais en général dans l’ordre des votes. Ce mois dernier j’ai travaillé sur l’utilisation du calendrier dans les scènes , demande qui avait 17 votes Cette demande n’a “que” 10 votes, donc soit il faut qu’il y ait plus de votes sur celle-ci, soit il faut attendre que toutes les demandes avec plus de votes soit développée
Hizo
Mars 7, 2022, 2:46
6
Mince, je trouve que ça serait bien pratique.
Peut être que ce petit up apportera d’autres votes
Encore merci pour tout ton taf !
1 « J'aime »
Bonjour à tous, j’ai commencé à travailler sur la feature. J’ai de bon résultat pour le moment. Il me reste à finir 2-3 truc et à faire les tests unitaire. Je pense avoir une PR d’ici la fin de la semaine
7 « J'aime »
La petite PR qui va bien :
GladysAssistant:master
← callemand:BoxDevices
opened 06:53AM - 31 May 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:
- [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.
- [ ] 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
Add a new box for the dashboard who can display a list of devices (like Device in room but without room)
<img width="1102" alt="Screenshot 2023-05-31 at 09 18 11" src="https://github.com/GladysAssistant/Gladys/assets/11317212/ae961e8b-62b0-4116-86e8-2e5cdee4eab1">
<img width="1233" alt="Screenshot 2023-05-31 at 09 17 58" src="https://github.com/GladysAssistant/Gladys/assets/11317212/e54c14a3-5ac6-44ed-99a1-3a8da5e880d2">
4 « J'aime »
Hizo
Mai 31, 2023, 9:10
10
Petite question, pourquoi limiter les types de materiel compatibles avec cette fonctionnalité ?
Merci! Je regarde demain et je te tiens au courant
On n’affiche que ce qu’on est capable d’afficher dans cette box, certaines fonctionnalités n’ont pas vocations à être affichée ici, exemple: les caméras !
Je viens de faire une review de la PR !
GladysAssistant:master
← callemand:BoxDevices
opened 06:53AM - 31 May 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:
- [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.
- [ ] 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
Add a new box for the dashboard who can display a list of devices (like Device in room but without room)
<img width="1102" alt="Screenshot 2023-05-31 at 09 18 11" src="https://github.com/GladysAssistant/Gladys/assets/11317212/ae961e8b-62b0-4116-86e8-2e5cdee4eab1">
<img width="1233" alt="Screenshot 2023-05-31 at 09 17 58" src="https://github.com/GladysAssistant/Gladys/assets/11317212/e54c14a3-5ac6-44ed-99a1-3a8da5e880d2">
2 « J'aime »
@pierre-gilles : J’ai fait des retours sur ta review
1 « J'aime »
Trop cool Je regarde ça lundi !
Je viens de te faire une review!
1 « J'aime »
Merci pour la review. J’ai fait les correctifs en conséquence
1 « J'aime »
Merci pour tes correctifs, ça marche de mon côté maintenant
J’ai quelques feedbacks mais rien de très sérieux. Au niveau des traductions je propose ça :
(Je pense que actuellement c’est un oubli, le texte est dupliqué 2 fois)
Tu trouveras la review complète ici : Add Devices box by callemand · Pull Request #1803 · GladysAssistant/Gladys · GitHub
Merci encore du dev
3 « J'aime »
@pierre-gilles : Normalement, j’ai fixé tous les « problèmes » que tu as remonté.
Est ce que tu pourrais refaire une petite review ?
1 « J'aime »
Merci pour les retours, les traductions par contre est-ce que tu peux mettre les mêmes que sur mon screenshot (placeholder et titre) ?
J’essaie de pousser l’utilisation du terme « widget » dans le front, et non pas « box » ce qui n’est pas très clair pour l’utilisateur
Lokkye
Juin 12, 2023, 11:44
20
C’est vrai que widget est plus clair que box. J’ai fait les changements