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] 📝 Révision du texte à propos de pix companion dans l'espace surveillant (PIX-17042) #11718

Merged

Conversation

yaf
Copy link
Member

@yaf yaf commented Mar 17, 2025

🥞 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.

image

🥓 Proposition

Modifier le contenu de la bannière d'information pour évoquer maintenant la documentation des surveillants.

image

🧃 Remarques

😋 Pour tester

Allez dans l'espace surveillant d'une session, vérifier que le message et le lien fonctionne

@yaf yaf added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-certif labels Mar 17, 2025
@yaf yaf self-assigned this Mar 17, 2025
@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 :

@yaf yaf changed the title [FEATURE] :memo2: Révision du texte à propos de pix companion dans l'espace surveillant (PIX-17042) [FEATURE] 📝 Révision du texte à propos de pix companion dans l'espace surveillant (PIX-17042) Mar 17, 2025
@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch 4 times, most recently from 69584ba to 723f1dc Compare March 17, 2025 14:31
@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch from 723f1dc to 30f2cdd Compare March 17, 2025 14:56
@yaf yaf requested a review from a team as a code owner March 18, 2025 09:20
@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch from 7b206c1 to fbffbe8 Compare March 18, 2025 09:31
Copy link
Member

@AndreiaPena AndreiaPena left a 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 :)

@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch from 7c4dcd1 to 1974bb9 Compare March 18, 2025 15:38
get pixCompanionDocumentationUrl() {
return 'https://cloud.pix.fr/s/fpeEyDpYEkMeqRX';
get invigilatorDocumentationUrl() {
return 'https://cloud.pix.fr/s/S5LHayrjbM4Zn5f';
Copy link
Contributor

@Steph0 Steph0 Mar 18, 2025

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)

Copy link
Member Author

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

@Steph0 Steph0 requested a review from Agnes-V March 18, 2025 16:28
@Agnes-V
Copy link
Contributor

Agnes-V commented Mar 19, 2025

Le lien fonctionne bien mais la dernière phrase du message n'est pas celle demandé par le métier.
Capture d’écran 2025-03-19 à 10 16 20
J'ai peur que la modification de wording donne l'impression au surveillant que s'il clique sur ce lien il ait accès à toute la documentation sur le rôle de surveillant (volumineuse) alors que le fichier ne concerne en réalité que la fonctionnalité de gestion de signalements pendant une session de certification. Le risque est que le surveillant se dise TLDR et zappe le doc. 🫠
Rappel du texte demandé par le métier : 🙏🏼

Important : Les problèmes techniques liés aux questions doivent uniquement être gérés pendant le test depuis l'interface du candidat et l’Espace Surveillant (ces signalements ne pourront pas être pris en compte a posteriori). Une documentation est disponible sur ce lien.

@yaf
Copy link
Member Author

yaf commented Mar 20, 2025

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.

@yaf
Copy link
Member Author

yaf commented Mar 20, 2025

@Agnes-V
Copy link
Contributor

Agnes-V commented Mar 24, 2025

@yaf hey re ! On a le retour du métier sur la modification de wording pour améliorer l'accessibilité.
Au lieu de "Une documentation est disponible sur ce lien." ils voudraient avoir "Documentation sur la gestion des signalements.."
Merci d'avoir remonté le point sur l'accessibilité 🙏🏼

@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch 3 times, most recently from 240ed58 to 1e19722 Compare March 24, 2025 13:45
@Agnes-V Agnes-V added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Mar 24, 2025
@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch from 1e19722 to 8580ec4 Compare March 24, 2025 15:54
@yaf yaf force-pushed the pix-17042-change-ES-pix-companion-info-banner branch from 8580ec4 to 07b818c Compare March 24, 2025 15:55
@pix-service-auto-merge pix-service-auto-merge merged commit 57e83c3 into dev Mar 24, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-17042-change-ES-pix-companion-info-banner branch March 24, 2025 16:04
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.

7 participants