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] Afficher la colonne "Écran de fin de test vu" sur la page de finalisation de session si le centre de certification n'utilise pas le portail surveillant (PIX-3748) #3931

Conversation

clemlatz
Copy link
Member

@clemlatz clemlatz commented Jan 11, 2022

🎄 Problème

Les fonctionnalités du nouvel espace surveillant permettent de s’assurer qu’un candidat passant son test de certification l’ai fait de A à Z sous surveillance. Il n’est donc plus nécessaire de signifier manuellement la fin de test d’un candidat pour le surveillant.

🎁 Solution

Sur la page de finalisation de session, retirer la colonne “Ecran de fin de test vu”

  • pour tous les types de CDC
  • si le CDC a accès au portail surveillant (est dans la whitelist)

🌟 Remarques

La case à cocher en question est implémentée à l'aide bouton sans libellé avec une icône pour indiquer son état ce qui est affreux 😱 d'un point de vue accessibilité, mais je me suis retenu de corriger vu que ce code est amené à disparaître rapidement.

🎅 Pour tester

Avec un centre qui n'a pas d'espace surveillant

  1. Se connecter à Pix Certif
  2. S'assurer que l'external ID du centre sélectionné n'est pas dans la var d'env END_TEST_SCREEN_REMOVAL_WHITELIST
  3. Accéder à une session finalisable et cliquer sur Finaliser
  4. Constater que la colonne "Écran de fin de test vu" apparaît

Avec un centre qui a un espace surveillant

  1. Se connecter à Pix Certif
  2. S'assurer que l'external ID du centre sélectionné est dans la var d'env END_TEST_SCREEN_REMOVAL_WHITELIST
  3. Accéder à une session finalisable et cliquer sur Finaliser
  4. Constater que la colonne "Écran de fin de test vu" n'apparaît pas

@clemlatz clemlatz self-assigned this Jan 11, 2022
@clemlatz clemlatz changed the title feat: hide end test column if feature toggle is enabled [FEATURE] Retirer la colonne "Écran de fin de test vu" sur la page de finalisation de session (PIX-3748) Jan 11, 2022
@pix-service
Copy link
Contributor

@clemlatz clemlatz force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from 2ce0417 to e7cdada Compare January 11, 2022 17:04
@clemlatz clemlatz added the ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase label Jan 11, 2022
@octo-topi octo-topi force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from e7cdada to cd0a9ef Compare January 12, 2022 10:41
@octo-topi octo-topi changed the title [FEATURE] Retirer la colonne "Écran de fin de test vu" sur la page de finalisation de session (PIX-3748) [FEATURE] Aficher la colonne "Écran de fin de test vu" sur la page de finalisation de session si le centre de certification n'utilise pas le portail surveillant (PIX-3748) Jan 12, 2022
@octo-topi octo-topi force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from cd0a9ef to 148e98f Compare January 12, 2022 10:44
@octo-topi octo-topi self-assigned this Jan 12, 2022
@octo-topi octo-topi force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch 2 times, most recently from 55b7522 to 1af19f5 Compare January 12, 2022 13:25
@clemlatz clemlatz changed the title [FEATURE] Aficher la colonne "Écran de fin de test vu" sur la page de finalisation de session si le centre de certification n'utilise pas le portail surveillant (PIX-3748) [FEATURE] Afficher la colonne "Écran de fin de test vu" sur la page de finalisation de session si le centre de certification n'utilise pas le portail surveillant (PIX-3748) Jan 12, 2022
@Libouk
Copy link
Member

Libouk commented Jan 13, 2022

Souci lors du test fonctionnel avec le compte certifsco pour le Centre SCO Collège des Anne-Étoiles. Après avoir rajouté dans Scalingo la variable END_TEST_SCREEN_REMOVAL_WHITELIST = 1 (id du centre en question) la colonne reste affichée

@clemlatz clemlatz force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch 3 times, most recently from 2bf9b69 to 80977b3 Compare January 13, 2022 17:23
@clemlatz
Copy link
Member Author

clemlatz commented Jan 13, 2022

Après avoir rajouté dans Scalingo la variable END_TEST_SCREEN_REMOVAL_WHITELIST = 1 (id du centre en question) la colonne reste affichée

Bien vu @Libouk ! Ça ne marchait pas parce qu'on comparait une String et un Number. Normalement, c'est corrigé, mais ça vaut le coup de refaire un test fonctionnel je pense (éventuellement avec plusieurs ids séparés par des virgules).

@clemlatz clemlatz force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch 2 times, most recently from f501a67 to 50947d2 Compare January 13, 2022 19:40
@aceol aceol force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from 50947d2 to db917d7 Compare January 14, 2022 14:16
@aceol aceol removed the ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase label Jan 14, 2022
@aceol
Copy link
Contributor

aceol commented Jan 14, 2022

Après avoir rajouté dans Scalingo la variable END_TEST_SCREEN_REMOVAL_WHITELIST = 1 (id du centre en question) la colonne reste affichée

Bien vu @Libouk ! Ça ne marchait pas parce qu'on comparait une String et un Number. Normalement, c'est corrigé, mais ça vaut le coup de refaire un test fonctionnel je pense (éventuellement avec plusieurs ids séparés par des virgules).

C'est OK 👍

@aceol aceol force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch 2 times, most recently from d9e7cf0 to be7779d Compare January 14, 2022 15:13
@octo-topi octo-topi added 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Jan 14, 2022
@octo-topi
Copy link
Contributor

Revue fonctionnelle OK en pair review

@aceol aceol force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from be7779d to d85af93 Compare January 14, 2022 15:31
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-3748-remove-end-test-column-on-session-finalize-page branch from d85af93 to 22aa40a Compare January 14, 2022 15:34
@pix-service-auto-merge pix-service-auto-merge deleted the pix-3748-remove-end-test-column-on-session-finalize-page branch January 14, 2022 15:41
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.

6 participants