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

Modèle de données : ajout de champs createdAt #566

Merged
merged 9 commits into from
Nov 2, 2022

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Oct 28, 2022

Actions effectuées :

  • j'ai ajouté un champ createdAt aux modèles suivants : Formation, Job, Shift, Commission et Event
  • le modèle Task a déjà le champ, et il est utilisé "autrement"...
  • les modèles Note, SwipeCard avaient déjà le champ (renommé created_at en createdAt)

Todo ?

  • j'ai préféré ne pas toucher aux modèles AnonymousBeneficiary, HelloassoPayment et Registration (il faudrait renommer created_at en createdAt)

@raphodn raphodn force-pushed the raphodn/entity-timestamps branch 3 times, most recently from c76db88 to f3cc270 Compare October 28, 2022 22:00
@raphodn
Copy link
Member Author

raphodn commented Oct 29, 2022

@petitalb trucs à prendre en compte :

  • pour les objets déjà créé, je ne sais pas comment ca remplit le champs 🤔 null je crois
  • pour les modèles AnonymousBeneficiary, HelloassoPayment et Registration je préfère le faire une autre fois, une fois qu'on se soit assuré que tout roule avec cette PR en prod :)

@raphodn raphodn merged commit 0769189 into master Nov 2, 2022
@raphodn raphodn deleted the raphodn/entity-timestamps branch November 2, 2022 19:17
@raphodn
Copy link
Member Author

raphodn commented Nov 2, 2022

@petitalb tu me diras si ca marche bien en fait, là j'ai une erreur quand je vais sur le profil d'un membre (en mode admin). erreur provenant du champ Note.created_at (que j'ai renommé en createdAt, ca ne marche peut-être pas ??)

@raphodn
Copy link
Member Author

raphodn commented Nov 2, 2022

réparé ici : e44e473

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

Successfully merging this pull request may close these issues.

Généraliser l'utilisation du champ createdAt
2 participants