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

[FEATURE] Ajouter la fonctionnalité d'import ONDE aux organisations ajoutées depuis le script de création en masse (PIX-12563). #9449

Merged

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Jul 4, 2024

🦄 Problème

Les écoles primaires (modélisées comme des organisations de type SCO-1D) doivent pouvoir importer leurs élèves via la fonctionnalité d'import ONDE.
Cette fonctionnalité n'est pas attribuée par défaut lors de l'import en masse d'organisations.
N'ayant pas la possibilité via une application de faire cette opération facilement, cette attribution post import est coûteuse et peut entraîner des erreurs.

🤖 Proposition

Afin d'éviter de devoir attribuer cette fonctionnalité à chaque école par la suite, on attribue la fonctionnalité d'import ONDE à toutes les organisations de type SCO-1D lors de l'import d'organisation en masse.

Pour cela, on utilise OrganizationForAdminRepository qui s'occupe déjà d'ajouter les fonctionnalités spécifiées par OrganizationForAdmin.
Le modèle OrganizationForAdmin attribue déjà l'import ONDE aux organisations de type SCO-1D.

🌈 Remarques

Il y a sûrement de la rationalisation ou de la clarification à faire dans l'usage des deux modèles et repository Organization et OrganizationForAdmin.

💯 Pour tester

Lancer le script d'import en masse avec une organisation SCO-1D.
Se rendre sur Pix Admin.
Sélectionner l'organisation importée et faire un import d'élèves via un fichier ONDE.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@HEYGUL HEYGUL force-pushed the pix-12563-add-onde-feature-to-mass-import-script branch 2 times, most recently from cf421af to 022e990 Compare July 4, 2024 13:00
@HEYGUL HEYGUL force-pushed the pix-12563-add-onde-feature-to-mass-import-script branch from 022e990 to 271b8af Compare July 7, 2024 06:35
Copy link
Contributor

@mariannebost mariannebost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tout me semble OK pour le scope Accès ✅

HEYGUL and others added 2 commits July 8, 2024 09:41
…ng them

Co-authored-by: Aurelie Crouillebois <aurelie.crouillebois@gmail.com>
Co-authored-by: <aurelie.crouilleboix@pix.fr>
Co-authored-by: <yaelle.marsiac@pix.fr>
Co-authored-by: <gwenvael.laskar@pix.fr>
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12563-add-onde-feature-to-mass-import-script branch from 271b8af to 07c619d Compare July 8, 2024 09:41
@pix-service-auto-merge pix-service-auto-merge merged commit 4b9972e into dev Jul 8, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12563-add-onde-feature-to-mass-import-script branch July 8, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants