Feat add UUID converter

Open plusiv opened this issue 1 year ago • 2 comments

Description

Use UUID Primary Keys instead of conventional int IDs. Until now the supported version of the standard is v4.

TODO

[x] Test changes
[x] Add documentation

Dec 28 '23 20:12 plusiv

@DannyFeliz I would like your review in this.

Happy new year in advance!

Dec 29 '23 15:12 plusiv

Hi @plusiv, I will try to take a close look at this PR before the end of the year.

Thank you very much for your contribution.

Dec 29 '23 18:12 DannyFeliz

Hi @DannyFeliz,

I hope you're doing well. Have you had time to review this?

Jul 08 '24 20:07 plusiv

@plusiv, in your opinion, what are the advantages of the UUID vs regular Int?

Jul 29 '24 23:07 DannyFeliz

Hi @DannyFeliz ,

To be honest, I am reconsidering this PR. The initial intent was to avoid sequential enumeration in the database. However, after evaluating the impact, I've realized that it doesn't make much sense given that this is public information, and there's nothing sensitive to conceal, such as a product database or user database. It's just information that everybody knows or could easily find out.

Jul 30 '24 04:07 plusiv

I guess we can ignore this PR; feel free to close it.

Jul 30 '24 04:07 DannyFeliz

Good! Closing it

Jul 30 '24 13:07 plusiv