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

Lier datasets et organisations #404

Closed
Tracked by #284
florimondmanca opened this issue Sep 1, 2022 · 0 comments · Fixed by #409
Closed
Tracked by #284

Lier datasets et organisations #404

florimondmanca opened this issue Sep 1, 2022 · 0 comments · Fixed by #409
Assignees

Comments

@florimondmanca
Copy link
Collaborator

florimondmanca commented Sep 1, 2022

Problème

Actuellement les formulaires de contribution et d'édition n'envoient pas le organizationSiret avec le jeu de données. Or on va en avoir besoin pour ensuite discriminer l'affichage selon l'organisation.

Solution envisagée

  • Modifier l'API pour insérer l'orga complète ({ name, siret }) dans dataset.catalog_record, car on a besoin du SIRET comme identifiant et du nom pour l'affichage
  • Modifier le type User pour ingérer user.organization_siret
  • Modifier le type CatalogRecord pour ingérer catalog_record.organization
  • Envoyer un organizationSiret dans le formulaire de contribution : pour la création, l'obtenir à partir de $user.organizationSiret ; pour l'édition, utiliser la valeur existante du jeu de données (faut-il l'envoyer à l'API ou est-ce pour affichage uniquement ?). Dans tous les cas, la valeur doit être en lecture seule.

NB : pour l'instant tout l'affichage se réfèrera à l'organisation factice. On peut très bien hardcoder le name comme "Ministère de la Culture". Il faudra ensuite prévoir une migration des données liées à l'orga factice vers une organisation cible unique, avant que plusieurs organisations ne soient présentes. Cf #389

Contexte supplémentaire

Extrait à partir de #399 : j'ai réalisé que ce serait un prérequis pour brancher ensuite les champs complémentaires

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

Successfully merging a pull request may close this issue.

1 participant