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

Champs meta_create_date et meta_update_date dans bib_organismes #215

Open
lpofredc opened this issue May 16, 2024 · 2 comments · May be fixed by PnX-SI/UsersHub-authentification-module#99

Comments

@lpofredc
Copy link
Contributor

La table bib_organismes ne contient aucun champ de métadonnée, a minima meta_create_date et meta_update_date.

Je propose une migration pour les ajouter à cette table.

Je privilégierai la maj par trigger (a priori géré le backend dans t_roles).

UsersHub/app/models.py

Lines 74 to 77 in 9100611

date_insert = db.Column(db.DateTime, default=db.func.now(), nullable=False)
date_update = db.Column(
db.DateTime, default=db.func.now(), nullable=False, onupdate=db.func.now()
)

Quelle piste privilégier pour la fonction de trigger:

  • Utiliser ou créer la fonction déjà présente dans public.fct_trg_meta_dates_change()?
  • Créer une fonction spécifique intégrée au schéma utilisateurs (utilisateurs.fct_trg_meta_dates_change())?
@camillemonchicourt
Copy link
Member

Oui ça serait bien d'avoir ces champs partout en effet.
A faire dans le dépôt UsersHub-authentification-module où sont gérées les migrations du schéma "utilisateurs".
Par contre il me semble que ça doit pas s'appuyer sur les fonctions existantes dans le schéma "public", car, sauf erreur de ma part, elles sont installées par GeoNature. Et UsersHub peut être installé tout seul sans GeoNature.

@lpofredc
Copy link
Contributor Author

Merci @camillemonchicourt pour ces précisions.

Je ferait donc une PR sur https://github.com/PnX-SI/UsersHub-authentification-module avec une fonction spécifique intégrée au schéma utilisateurs.

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 a pull request may close this issue.

2 participants