Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

uniformare gli id #52

Open
dinghino opened this issue Apr 8, 2017 · 1 comment
Open

uniformare gli id #52

dinghino opened this issue Apr 8, 2017 · 1 comment

Comments

@dinghino
Copy link
Contributor

dinghino commented Apr 8, 2017

considerando che tutti i modelli esposti (User, Order ed Item) hanno un field che contiene un uuid4, pensavo potremmo uniformare il loro nome e la loro creazione all'interno di BaseModel, in maniera simile a come è stato fatto per created_at e updated_at
L'idea sarebbe di sovrascrivere il field id che genera peewee in automatico, facendogli usare un uuid invece di un valore progressivo, In questo modo ovvieremo al dover importare ed utilizzare il modulo uuid ogni volta che dobbiamo creare una nuova istanza di un qualsiasi modello, avremmo un nome uniforme e non ridondante per gli id (item.item_id, order.order_id, ...), e risolveremmo i problemi attualmente presenti con jsonapi

@dinghino dinghino mentioned this issue Apr 9, 2017
@marcotinacci
Copy link

marcotinacci commented May 9, 2017

è stata appena mergiata una mia pull request #104 che uniforma i nomi degli id in tutto il progetto, spostare l'uuid nella classe base sarebbe possibile ma forzerebbe tutti i modelli ad averne uno anche quando non necessario (come per il modello OrderItem). @gvaldambrini @GendoIkari ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants