-
Notifications
You must be signed in to change notification settings - Fork 53
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] Mise à jour du wording lors du défocus pour candidats avec besoins d'ajustements (PIX-14428). #10258
Merged
alexandrecoin
merged 12 commits into
dev
from
pix-14428-update-unfocused-messages-in-certif
Oct 9, 2024
Merged
[FEATURE] Mise à jour du wording lors du défocus pour candidats avec besoins d'ajustements (PIX-14428). #10258
alexandrecoin
merged 12 commits into
dev
from
pix-14428-update-unfocused-messages-in-certif
Oct 9, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
alexandrecoin
force-pushed
the
pix-14428-update-unfocused-messages-in-certif
branch
7 times, most recently
from
October 8, 2024 04:05
3c69d45
to
a666f5e
Compare
alexandrecoin
added
👀 Tech Review Needed
👀 Func Review Needed
and removed
Development in progress
labels
Oct 8, 2024
alexandrecoin
force-pushed
the
pix-14428-update-unfocused-messages-in-certif
branch
from
October 8, 2024 06:56
a666f5e
to
643a484
Compare
mcampourcy
added
Func Review OK
PO validated functionally the PR
and removed
👀 Func Review Needed
labels
Oct 8, 2024
mcampourcy
approved these changes
Oct 8, 2024
mcampourcy
reviewed
Oct 8, 2024
Steph0
approved these changes
Oct 9, 2024
api/src/certification/shared/domain/models/CertificationCourse.js
Outdated
Show resolved
Hide resolved
api/src/certification/shared/infrastructure/repositories/certification-course-repository.js
Outdated
Show resolved
Hide resolved
api/src/certification/shared/infrastructure/repositories/certification-course-repository.js
Outdated
Show resolved
Hide resolved
alexandrecoin
force-pushed
the
pix-14428-update-unfocused-messages-in-certif
branch
from
October 9, 2024 09:08
643a484
to
ad2f3b4
Compare
alexandrecoin
force-pushed
the
pix-14428-update-unfocused-messages-in-certif
branch
from
October 9, 2024 09:33
ad2f3b4
to
c09e026
Compare
alexandrecoin
force-pushed
the
pix-14428-update-unfocused-messages-in-certif
branch
from
October 9, 2024 09:36
c09e026
to
eafbce3
Compare
alexandrecoin
deleted the
pix-14428-update-unfocused-messages-in-certif
branch
October 9, 2024 09:41
dianeCdrPix
pushed a commit
that referenced
this pull request
Oct 9, 2024
…besoins d'ajustements (PIX-14428). (#10258) Co-authored-by: Guillaume Lagorce <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Dans le cadre de l’epix a11y , une version “accessible” des focus va être implémentée.
Si on permet à un candidat ayant un test aménagé a11y de valider une question focus bien répondue, même en cas de défocus, le message en cas de défocus actuel est trompeur.
🤖 Proposition
Mise à à jour du wording des warning des questions focus pour un test aménagé a11y : pour un test de certification v3, pour lequel la case “aménagement a11y a été coché” :
Lorsque le candidat sort de la zone de question
Nouveau wording :
🇫🇷 “Répondez à cette question sans vous aider d’internet ou d’applications.”
🇬🇧 “Answer this question without using the Internet or any applications.”
Lorsque le candidat défocus de l'épreuve
Nouveau wording :
🇫🇷 “Nous avons détecté un changement de page. Si vous avez été contraint de changer de page pour utiliser un outil d’accessibilité numérique (tel qu'un lecteur d'écran ou un clavier virtuel), répondez tout de même à la question.”
🇬🇧 “We have detected a page switch. If you have been forced to change pages in order to use a digital accessibility tool (such as a screen reader or virtual keyboard), please answer the question anyways.”
🌈 Remarques
On décide de renvoyer
undefined
dans le payload du serializer pour le mois à venir avant la généralisation.💯 Pour tester
Créer une session de certification V3 dans un centre featurisé a11y (
[email protected]
)Ajouter deux candidats à cette session
Modifier les infos d'un des candidats afin d'y ajouter un besoin d'ajustement
Passer les sessions des deux candidats et vérifier que les messages de défocus lors des épreuves focus correspondent aux différents besoins d'ajustements des candidats
Créer une session V2 et y ajouter un candidat
Passer le test de certification et vérifier que les messages de défocus lors des épreuves focus n'ont pas changé