-
Notifications
You must be signed in to change notification settings - Fork 24
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
style(models/User.public_id): add help_text #4397
Conversation
8999756
to
af9f1d6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je l'aurais imaginé au sein de l'autre PR (car c'est à cause de l'autre PR que cet index est nécessaire), mais ça peut être une PR préparatoire.
Idéalement il faudrait dire dans le message de commit que c'est pour préparer le prochain ticket où l'on va beaucoup requêter les job seeker par public_id.
model_name="user", | ||
name="public_id", | ||
field=models.UUIDField( | ||
db_index=True, default=uuid.uuid4, unique=True, verbose_name="identifiant public opaque, pour les API" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu pourrais au passage en profiter pour mettre à jour le verbose_name "pour les API et les URLs publiques"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai fais :
verbose_name="identifiant public",
help_text="identifiant opaque, pour les API et les URLs publiques",
Ça te va ? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En fait je me rend compte un peu tardivement que cela ne sert pas car l'index existe déjà via la contrainte d'unicité.
On peut néanmoins garder le changement de verbose_name en l'intégrant dans la migration 0008.
fix: code style feat(User): added help_text to public_id field fix(User): remove db_index which is implicit
b19010e
to
2ee28bb
Compare
Ah oui t'as raison (https://docs.djangoproject.com/en/5.0/ref/models/fields/#unique) OK, merci - j'ai retiré l'addition de |
🤔 Pourquoi ?
Accès au Utilisateur par
public_id
plus efficace🚨 À vérifier