Salut à tous,
Petite proposition, je sais qu’il arrive qu’il y ait des collisions avec les selectors quand on créé un device dans une intégration qui a le même nom qu’un device dans une autre intégration.
Je propose d’ajouter un nombre à 4 chiffres à la fin de chaque selector futur (pas ceux déjà créé), pour éviter les collisions MAIS garder un selector lisible et “human readable”
Exemple:
Nom: Capteur température chambre
Selector: capteur-temperature-chambre-1234
Le selector est tout autant lisible, mais pour autant plus de collisions possibles.
2 « J'aime »
Pour moi ça fait sens, c’est la même problematique d’ailleurs que pour les noms de scène ou de dashboard.
1 « J'aime »
VonOx
Octobre 22, 2021, 6:18
3
Du même avis que @lmilcent
Désolé @pierre-gilles j’avais pas fait le rapprochement avec l’issue sur github
1 « J'aime »
Top merci pour vous retours
Draft PR:
GladysAssistant:master ← GladysAssistant:add-random-string-to-selector
opened 10:35AM - 22 Oct 21 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)
- [ ] 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.json`) 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
Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!
(C’est encore WIP, encore beaucoup de testing à faire)
Je n’avais pas compris jusqu’à présent a quoi cela servait.
Cela permettrait d’avoir des dashboards avec le même nom sur plusieurs utilisateurs.
