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

[BUGFIX] Petites corrections pour faire fonctionner l'import ONDE (Pix-12275) #8730

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

yaelle6
Copy link
Contributor

@yaelle6 yaelle6 commented Apr 23, 2024

🦄 Problème

Suite à un test fonctionnel de l’import ONDE, on s’est rendu compte que:

  • Il y a une faute de wording pour récupérer organizationLearnerImportFormatId
  • On n'était pas iso: pour les headers on regardait l'attribut Is Required alors que pour les valeurs de colonnes c'était required.
  • On avait mis les 3 usecases,qu'on appelle pour l'import, dans la même transaction donc quand une erreur survenait, on faisait un roll back complet et donc la mise à jour de l'organization-import avec le statut en erreur ne pouvait avoir lieu car l'organization-import avait été supprimé.
  • Il manquait des choses au niveau des seeds:
    • accepted_encodings, les required au niveau des formats

🤖 Proposition

Corriger les soucis

🌈 Remarques

💯 Pour tester

  • Vérifier que tous les tests passent
    ###Si vous voulez faire une review fonctionnelle
  • Aller sur pix Orga et se connecter avec 1d-orga@example.net et récupérer le Bearer
  • Récupérer un exemple d'import ONDE
  • Se connecter à l'api via scalingo via la commande suivante:
curl -i '<URL_API_RA/api/organizations/9000/import-organization-learners' -H "Authorization: Bearer <TOKEN>" --data-binary '@PATH_FICHIER_ONDE'

Et aller voir en BDD si pour l'orga id 9000, les 2 learners ont été créé et que les autres ont été désactivés.

@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 :

@yaelle6 yaelle6 force-pushed the pix-12275-fix-bug-to-ONDE-import branch from d0b2d42 to f24cf11 Compare April 24, 2024 08:59
@yaelle6 yaelle6 changed the base branch from dev to pix-12134-use-intern-api-to-retrieve-pix1d--learners April 24, 2024 09:00
@yaelle6 yaelle6 requested a review from a team as a April 24, 2024 09:00
@yaelle6 yaelle6 changed the base branch from pix-12134-use-intern-api-to-retrieve-pix1d--learners to dev April 24, 2024 09:00
@yaelle6 yaelle6 closed this Apr 24, 2024
@yaelle6 yaelle6 reopened this Apr 24, 2024
@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 :

@yaelle6 yaelle6 force-pushed the pix-12275-fix-bug-to-ONDE-import branch from f24cf11 to fe41ddf Compare April 24, 2024 13:23
@lionelB lionelB changed the base branch from dev to pix-12134-use-intern-api-to-retrieve-pix1d--learners April 24, 2024 13:41
@lionelB lionelB changed the base branch from pix-12134-use-intern-api-to-retrieve-pix1d--learners to dev April 24, 2024 13:42
@pix-service-auto-merge pix-service-auto-merge merged commit d3b5d48 into dev Apr 24, 2024
5 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12275-fix-bug-to-ONDE-import branch April 24, 2024 14:58
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.

5 participants