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

Nettoyage de l'organisation "legacy" #389

Closed
4 of 5 tasks
Tracked by #122
florimondmanca opened this issue Aug 29, 2022 · 5 comments · Fixed by #489
Closed
4 of 5 tasks
Tracked by #122

Nettoyage de l'organisation "legacy" #389

florimondmanca opened this issue Aug 29, 2022 · 5 comments · Fixed by #489

Comments

@florimondmanca
Copy link
Collaborator

florimondmanca commented Aug 29, 2022

Actuellement, on utilise une organisation dite "legacy" dont le SIRET est 000 000 000 00000 comme "placeholder" : les jeux de données sont rattachés à cette organisation par défaut. Cela a été ajouté dans #351.

Pour pouvoir clore #122, il faut que les fonctionnalités utilisent SIRET explicite partout où c'est nécessaire. Exemples : ajouter un dataset => SIRET nécessaire pour savoir dans quel catalogue l'ajouter ; ajouter un utilisateur => SIRET nécessaire pour savoir à quelle orga l'associer, etc.

Dans le cadre du milestone, l'organisation "legacy" continuera d'exister, mais et accueillera seulement les utilisateurs admin.

Critères d'acceptation

@johanricher
Copy link
Member

Comme je disais ici : #334 (comment)

A court terme (livraison 13 octobre) est-ce que c'est un must have de supprimer l'organisation factice (que j'appelle aussi "legacy") ? Celle-ci va devenir invisible (car le contenu de son catalogue supprimé) et seuls les super administrateurs (#288) en feront partie. L'alternative serait de créer plusieurs nouvelles organisations (multi, Fairness, DINUM...) auxquelles les super administrateurs devront appartenir, ça peut être envisagé comme un "should have" mais je ne vois pas d'urgence à cela.

Qu'en pensez-vous ?

@florimondmanca
Copy link
Collaborator Author

Oui vu la discussion que l'on a eue, on peut retravailler ce ticket en "Nettoyer l'organisation factice".

Mon objectif était surtout de ne plus faire référence à cette organisation dans le code. Elle pourra exister dans un environnement donné mais on ne doit plus nulle part y ajouter de données (à ce stade elle est encore utilisée comme organisation par défaut dans beaucoup d'opérations du back).

@florimondmanca florimondmanca changed the title Retirer l'organisation factice Nettoyage de l'organisation "legacy" Sep 21, 2022
@florimondmanca florimondmanca mentioned this issue Sep 26, 2022
4 tasks
@florimondmanca
Copy link
Collaborator Author

@johanricher @DaFrenchFrog Il y a ce critère d'acceptation (proposé par moi)

L'organisation "legacy" n'apparaît pas dans les choix du filtre "Catalogue"

Qu'en pensez-vous ? L'"Organisation par défaut" apparaît actuellement dans le filtre "Catalogue", est-ce un problème ? Je pensais faire une simple exception dans le code à ce stade pour exclure cette orga des choix du filtre.

@DaFrenchFrog
Copy link
Collaborator

Je suis pour 👍

@johanricher
Copy link
Member

Ok 👍

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.

3 participants