# Proposition: Ajouter un nombre aléatoire à 4 chiffres à tous les "selector" pour éviter les collisions

**URL:** <https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617>\
**Category:** Développement\
**Created:** [Octobre 18, 2021, 7:54 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617 "2021-10-18T07:54:20Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Octobre 18, 2021, 7:54 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/1 "2021-10-18T07:54:21Z")

</div>

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.

---

<div class="post-metadata">

**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\
**Post date:** [Octobre 22, 2021, 5:56 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/2 "2021-10-22T05:56:35Z")

</div>

Pour moi ça fait sens, c’est la même problematique d’ailleurs que pour les noms de scène ou de dashboard.

---

<div class="post-metadata">

**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\
**Post date:** [Octobre 22, 2021, 6:18 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/3 "2021-10-22T06:18:35Z")

</div>

Du même avis que @lmilcent

Désolé @pierre-gilles j’avais pas fait le rapprochement avec l’issue sur github 😫

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Octobre 22, 2021, 6:19 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/4 "2021-10-22T06:19:33Z")

</div>

Top merci pour vous retours 🙂

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Octobre 22, 2021, 10:36 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/5 "2021-10-22T10:36:22Z")

</div>

Draft PR:

> <https://github.com/GladysAssistant/Gladys/pull/1336>
>
> \### 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)

---

<div class="post-metadata">

**Author:** ![Tlse-vins](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/tlse-vins/32/6054_2.png) [@Tlse-vins](https://community.gladysassistant.com/u/Tlse-vins)\
**Post date:** [Mai 22, 2022, 8:34 UTC](https://community.gladysassistant.com/t/proposition-ajouter-un-nombre-aleatoire-a-4-chiffres-a-tous-les-selector-pour-eviter-les-collisions/6617/6 "2022-05-22T08:34:59Z")

</div>

Je n’avais pas compris jusqu’à présent a quoi cela servait.  
Et comme je suis dans ce cas de figure. Je suis d’accord avec ta proposition.

Cela permettrait d’avoir des dashboards avec le même nom sur plusieurs utilisateurs.
