Skip to content
This repository has been archived by the owner on May 4, 2020. It is now read-only.

Changer les id en hashid #80

Open
ThomasLachaux opened this issue Oct 4, 2019 · 3 comments
Open

Changer les id en hashid #80

ThomasLachaux opened this issue Oct 4, 2019 · 3 comments
Labels
année prochaine Fonctionnalité à faire l'an prochain optimisation Les performances ou le code peut être optimisé

Comments

@ThomasLachaux
Copy link
Member

ThomasLachaux commented Oct 4, 2019 •
edited
Loading

Avantages: plus de lisibilité
https://www.npmjs.com/package/hashids
@ThomasLachaux ThomasLachaux added the optimisation Les performances ou le code peut être optimisé label Oct 4, 2019
@nyc4m
Copy link

nyc4m commented Nov 5, 2019

On garde ça ou pas ? 👀

@Fleuh
Copy link
Member

Fleuh commented Nov 5, 2019 •
edited
Loading

Surtout pas, un random a BEAUCOUP plus de chances de générer un doublon qu'un UUID (pour ne pas dire qu'il y a absolument aucune chance de générer deux fois le même UUID). D'autre part la génération d'UUID est gérée directement en externe pourquoi se faire chier à changer pour un truc manuel / pseudo manuel ?

@ThomasLachaux
Copy link
Member Author

Les barcodes sont générés avec un random ^^.

Sinon, go hashid ^^

@ThomasLachaux ThomasLachaux changed the title Changer les ID de UUID à un random 10^12 Changer les id en hashid Nov 7, 2019
@ThomasLachaux ThomasLachaux added the année prochaine Fonctionnalité à faire l'an prochain label Nov 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
année prochaine Fonctionnalité à faire l'an prochain optimisation Les performances ou le code peut être optimisé
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nyc4m @Fleuh @ThomasLachaux