-
Notifications
You must be signed in to change notification settings - Fork 58
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] 📝 Révision du texte à propos de pix companion dans l'espace surveillant (PIX-17042) #11718
[FEATURE] 📝 Révision du texte à propos de pix companion dans l'espace surveillant (PIX-17042) #11718
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
69584ba
to
723f1dc
Compare
723f1dc
to
30f2cdd
Compare
7b206c1
to
fbffbe8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
un loupé sur les commits visiblement mais sinon c'est bon pour moi :)
7c4dcd1
to
1974bb9
Compare
get pixCompanionDocumentationUrl() { | ||
return 'https://cloud.pix.fr/s/fpeEyDpYEkMeqRX'; | ||
get invigilatorDocumentationUrl() { | ||
return 'https://cloud.pix.fr/s/S5LHayrjbM4Zn5f'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proposition : les urls externes (donc hors pix.xxx) sans logique de construction devraient etre en cle de traduction et leur mise a jour devrait etre a la main Produit/Metier (via Phrase)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ça semble un peu bricolage quand même... Même si je comprends l'idée. Là, c'est pas hors pix d'ailleurs
Le lien fonctionne bien mais la dernière phrase du message n'est pas celle demandé par le métier.
|
Pour partager ici ce que nous nous sommes dit ce matin : avec un label « lien » sur un lien vers un document n'est pas très bon niveau accessibilité... C'est pour ça que j'avais changé la tournure. Discutons avec le métier pour trouver une bonne tournure qui soit accessible. |
J'ajoute la ref RGAA https://accessibilite.numerique.gouv.fr/methode/criteres-et-tests/#6.1 |
@yaf hey re ! On a le retour du métier sur la modification de wording pour améliorer l'accessibilité. |
240ed58
to
1e19722
Compare
1e19722
to
8580ec4
Compare
Co-authored-by: Andreia Pena <58915422+AndreiaPena@users.noreply.github.com>
8580ec4
to
07b818c
Compare
🥞 Problème
Les signalements à traiter pendant la session sont une nouveauté de la certification v3, les surveillants ne se rendent pas toujours compte qu’ils doivent obligatoirement les prendre en compte pendant la session.
🥓 Proposition
Modifier le contenu de la bannière d'information pour évoquer maintenant la documentation des surveillants.
🧃 Remarques
😋 Pour tester
Allez dans l'espace surveillant d'une session, vérifier que le message et le lien fonctionne