VonOx
September 4, 2021, 9:03am
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
March 7, 2022, 11:04am
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
March 7, 2022, 2:46pm
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 Like
Hello everyone, I’ve started working on the feature. I have good results so far. I have 2-3 things left to finish and need to do the unit tests. I think I’ll have a PR by the end of the week.
7 Likes
The little PR that does the trick:
master ← callemand:BoxDevices
ouvert 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 Likes
Hizo
May 31, 2023, 9:10pm
10
Quick question, why limit the types of equipment compatible with this feature?
Thanks! I’ll look tomorrow and keep you posted
We only display what we’re able to show in this box; some features are not meant to be displayed here, for example: cameras!
Lokkye
June 2, 2023, 1:48pm
13
@pierre-gilles : I left feedback on your review
1 Like
So cool I’ll check that out on Monday!
I just left you a review!
1 Like
Lokkye
June 7, 2023, 4:39pm
16
Thanks for the review. I’ve made the corrections accordingly
1 Like
Thanks for your fixes, it’s working on my side now
I have some feedback but nothing very serious. Regarding the translations I suggest this :
(I think it’s currently an oversight, the text is duplicated twice)
You will find the full review here : https://github.com/GladysAssistant/Gladys/pull/1803#pullrequestreview-1471210131
Thanks again for the dev work
3 Likes
Lokkye
June 12, 2023, 9:53am
18
@pierre-gilles : Normally, I have fixed all the « problems » you reported.
Could you do a quick review again?
1 Like
Thanks for the feedback — for the translations, could you use the same ones as in my screenshot (placeholder and title)?
I’m trying to promote the use of the term « widget » in the front
Lokkye
June 12, 2023, 11:44am
20
It’s true that widget is clearer than box. I’ve made the changes