guim31
Décembre 13, 2022, 12:04
1
Actuellement dans le cadre de l’utilisation de boutons multiples connectés (Xiaomi ou autres) on ne peut pas les intégrer directement dans les scènes, car lorsqu’on leur appuie dessus, ce qui remonte c’est des choses du genre :
single_left, single_right, single_both, double_left, double_right, double_both, triple_left, triple_right, triple_both, hold_left, hold_right, hold_both
Or dans les scènes, nous ne pouvons utiliser que ça :
Qui n’autorise que des entiers numériques.
Il faudrait donc pouvoir intégrer ces valeurs « textuelles » dans les scènes, sans passer par Nodered.
VonOx
Décembre 13, 2022, 1:15
2
La PR est déjà en cours par @AlexTrovato
GladysAssistant:master
← atrovato:button-click-user-friendly
opened 10:15AM - 04 Nov 22 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?~~
- [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)
- [ ] 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).~~
- [x] 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
Override button click values by user friendly wording.
See https://community.gladysassistant.com/t/affichage-des-boutons-textuels-dans-les-scenes-et-tableau-de-bord-au-lieu-dentier/7644
#### Dashboard
![image](https://user-images.githubusercontent.com/1839717/199950039-b56e0713-ac05-44b9-98cf-4871518a116a.png)
#### Scene
![image](https://user-images.githubusercontent.com/1839717/199949729-c2f44f48-736e-4b49-842e-bad744f2d92d.png)
2 « J'aime »
guim31
Décembre 13, 2022, 1:44
3
Damn… il faudrait que je visite Github plus souvent ! (sous réserve que j’y comprenne quelque chose !)
@guim31 ,
on est d’accord qu’il s’agit de la même demande que tu avais créé en novembre 2022 :
Affichage des boutons textuels dans les scènes et tableau de bord (au lieu d'entier) ?
Comme tu l’as passé sur « résolue » je me permet de la fermer. N’hésite pas à me dire si il y a besoin de la rouvrir, la PR (qui me semble est commune avec ton autre demande) n’étant pas encore sortie.
1 « J'aime »