[TECH] Export de la liste blanche SCO #10593

Steph0 · 2024-11-19T16:07:57Z

🍂 Problème

Dans la suite des episodes "on en peut plus de gerer la liste blanche par variable d'environnement", et pour pouvoir importer une liste blanche, il faut pouvoir aussi l'exporter.

🌰 Proposition

Creer un bouton d'export
Compatibilite entre le fichier d'export et d'import

🎃 Remarques

J'ai ajoute aussi une petite securite sur l'import pour ne pas pouvoir importer une liste invalide

🪵 Pour tester

Sur Pix Admin, aller dans Administration > Certification > Liste blanche centres SCO
- soit /administration/certification
Cliquer sur le bouton d'export
- Pas d'erreur, et recuperation d'un fichier CSV avec un en-tete "externalId"
Reperer, ou creer, un centre SCO, et y mettre un externalId
- Par exemple CERTIFICATION_SCO_MANAGING_STUDENTS_EXTERNAL_ID est un bon candidat
Editer le fichier CSV export, y mettre l'externalId
Cliquer sur le bouton d'import et importer le fichier sauvegarde
- Pas d'erreur
Refaire un export, et voir que le nouveau fichier d'export contient l'externalId
Editer le fichier CSV export, y mettre un external id "faux", genre "PAS_MOI"
Tenter d'exporter un fichier avec ce mauvais id
- Constater un message d'erreur
- Tenter un export et voir que la liste blanche n'a pas bouge (on a refuse la liste avec des data invalides)

pix-bot-github · 2024-11-19T16:08:47Z

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

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

AndreiaPena · 2024-11-19T16:50:40Z

admin/app/components/administration/certification/sco-whitelist-configuration.gjs

+      const token = this.session.data.authenticated.access_token;
+      await this.fileSaver.save({ url, fileName, token });
+    } catch (error) {
+      this.notifications.error(this.intl.t('pages.administration.certification.sco-whitelist.export.error'));


si jamais ma PR passe avant (mais j'y crois moyen 😅)

Suggested change

this.notifications.error(this.intl.t('pages.administration.certification.sco-whitelist.export.error'));

this.pixToast.sendErrorNotification({message:this.intl.t('pages.administration.certification.sco-whitelist.export.error')});

Est-ce que je peux le faire avant que ta PR merge ? Sinon je vais devoir laisser ainsi

Tu pourrais avant mais la notif n'apparaîtra pas tant que ma PR n'est pas mergé :)

admin/translations/fr.json

api/src/certification/configuration/application/sco-whitelist-route.js

admin/app/styles/app.scss

AndreiaPena · 2024-11-19T17:00:39Z

Question : Avec l'ajout de cette PR, l'import/export de la whitelist est terminé, non ?

Si oui il y a un bandeau Feature en cours à retirer coté front :)

Sinon, pas terminé la review, je poursuis demain :)

Co-authored-by: P-Jeremy < [email protected]>

Steph0 · 2024-11-20T08:34:11Z

Question : Avec l'ajout de cette PR, l'import/export de la whitelist est terminé, non ?

Si oui il y a un bandeau Feature en cours à retirer coté front :)

Sinon, pas terminé la review, je poursuis demain :)

C'est fait, j'ai retire le bandeau car la partie Pix Admin est en effet finie.
La feature dans sa totalite n'est par contre pas encore terminee, il me reste a maintenant brancher l'usage de la nouvelle colone en lieu et place de la variable d'environnement (mais ce sera un prochain tech times)

AndreiaPena · 2024-11-20T09:39:37Z

Quand je télécharge, j'ai l'entête "externalId" ✅
Puis je renseigne un externalId (EX123, l'externalId du Centre de certif Contenu ) sous l'entête, ça échoue ✅
=> c'est un externalId PRO
=> Je suis pas bien réveillé du coup j'ai eu du mal à comprendre pourquoi il ne passait pas à l'origine. Je me demande si on ne développerait pas le message d'erreur (à voir si cette fonctionnalité mérite qu'on développe un peu plus finement les erreurs)
Puis je réessaye avec CERTIFICATION_SCO_MANAGING_STUDENTS_EXTERNAL_ID ça fonctionne ✅
J'exporte la liste, je le retrouve bien dedans ✅
J'importe une nouvelle liste avec plusieurs externalIds SCO uniquement, la notif me dit que c'est bon ✅

"externalId"
"CERTIFICATION_SCO_MANAGING_STUDENTS_EXTERNAL_ID","SCOSCO","SCO_123"

J'exporte de nouveau la liste, je ne trouve pas mes 3 externalIds, uniquement le premier 🔴
Je recommence en me disant que j'ai probablement mal renseigné les ids

"externalId"
"CERTIFICATION_SCO_MANAGING_STUDENTS_EXTERNAL_ID"
"SCOSCO"
"SCO_123"

J'exporte de nouveau la liste et j'ai les 3 ✅

AndreiaPena · 2024-11-20T09:58:26Z

Simulé une erreur lors de l'export, j'ai bien la notif d'erreur ✅