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

[TECH] Supprimer l'ancienne route de génération d'identifiants et mots de passe en masse (PIX-14498) #10178

Merged

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Sep 25, 2024

🦄 Problème

Actuellement l’API de génération d’identifiants et mots de passe est disponible sur 2 routes différentes

  • /api/sco-organization-learners/password-reset
  • api/sco-organization-learners/batch-username-password-generate

Ceci afin que les modifications effectuées sur la PR #10096 soient rétrocompatibles avec les utilisateurs de Pix Orga connectées sur une ancienne version de l’API Pix.

🤖 Proposition

Supprimer la route /api/sco-organization-learners/password-reset

🌈 Remarques

La description de l'api a été aussi améliorée sur cette PR

💯 Pour tester

Test de non régression

  • Se connecter à la RA de Pix Orga avec le compte [email protected]
  • Sélectionner, en cliquant sur le profil en haut à droite, l'organisation Collège House of The Dragon
  • Aller dans l'onglet Elèves
  • Sélectionner / Cocher les élèves Mikasa, Eliza, Edward, Hermione et Bob
  • Vérifier que le bouton Réinitialiser les mots de passe des élèves sélectionnés dans la barre en bas s'affiche bien.
  • Cliquer sur le bouton
  • Vérifier que la modale s'affiche bien avec le fait que 2 élèves auront leur mot de passe réinitialisé.
  • Ouvrir la console navigateur
  • Cliquer sur Confirmer
  • Vérifier que l'appel /api/sco-organization-learners/batch-username-password-generate renvoie bien une 200.
  • Vérifier, dans le contenu du fichier CSV, qu'on retrouve bien les informations de Eliza et Bob

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

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.

Lu et testé avec succès sur Chrome ✅

@er-lim er-lim force-pushed the pix-14498-remove-old-batch-username-password-generate-route branch from 093dc21 to 16e6207 Compare October 1, 2024 07:46
@er-lim er-lim enabled auto-merge (squash) October 1, 2024 07:49
@er-lim er-lim force-pushed the pix-14498-remove-old-batch-username-password-generate-route branch 2 times, most recently from 48b9c19 to 5c5c055 Compare October 1, 2024 08:02
@er-lim er-lim force-pushed the pix-14498-remove-old-batch-username-password-generate-route branch from 5c5c055 to 7186ddc Compare October 1, 2024 08:12
@er-lim er-lim merged commit 69bf28d into dev Oct 1, 2024
7 of 8 checks passed
@er-lim er-lim deleted the pix-14498-remove-old-batch-username-password-generate-route branch October 1, 2024 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Func Review OK PO validated functionally the PR team-acces Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants