[FEAT] Affichage des commentaires auto jury ou commentaires manuels si absents (PIX-10528).

[Steph0]

:unicorn: Problème

Pour l'affichage des commentaires jury, s’il n’y a rien dans la nouvelle colonne commentByAutoJury, il faut que les anciens commentaires stockés dans les anciennes colonnes à savoir commentForCandidate et commentForOrganization soient toujours visibles dans Pix Admin, Pix App et Pix Orga.

:robot: Proposition

• Créer un mécanisme qui bascule automatiquement par défaut sur les anciens commentaires manuels quand le commentaire automatique n'est pas renseigné
• Sur les commentaires automatique, prendre en charge l'internationalisation

:rainbow: Remarques

• On a créé d'avance deux clés FRAUD et CANCELLED_NEUTRALIZATION, car on avait besoin de valeurs pour les tests

  • Ce sont les bons textes qui ont été mis dans cette PR

• Sur test : Affichage des commentaires jury, si commentaire AUTO, comme l'API de sauvegarde ne renvoie pas la mise à jour, on doit rafraichir la page. La modif de ce comportement semble pour nous être de l'ordre des autres tickets (PIX-10538 en l'occurence)

:100: Pour tester

Résultats pour une classe (SCO)

• Dans Pix Orga avec [email protected]
  • Télécharger la liste des résultats pour la classe "Terminale C"
  • Vérifier les commentaires organisation, voir que dans un cas on a le commentaire "auto" qui s'affiche (dans la bonne langue) et que pour l'autre cas c'est le commentaire manuel
  • Rééessayer sur une autre classe, par exemple "2nde B", qui n'a pas de commentaire et voir que la colonne est vide

Affichage des commentaires jury

• Dans Pix Admin, avec [email protected]
  • Sur la certifcation 142088 éditer les commentaires manuels, enregistrer
  • Rafraichir la page, vérifier que les commentaires enregistrés apparaissent bien
  • Vous pouvez éventuellement rejouer le test précèdent sur Pix Orga pour voir la mise à jour
  • En revanche sur la certif 131740, essayez de modifier les commentaires Jury, enregistrez, rafraichir la page
  • Voir que même si l'enregistrement a pu avoir lieu, pour autant au refresh c'est bien toujours le AUTO qui prends le pas

Résultats d'une session

• Dans Pix Admin, avec [email protected]
  • Sur la session 7401, récupérer le lien de téléchargement des résultats
  • Télécharger les résultats et vérifier les commentaires organisation, ils doivent être cohérents avec les tests précèdents !

Résultats pour le candidat

• Dans Pix App, avec [email protected]
• Dans Profil > Mes Certifications > Voir mes résultats
  • Vérifier que les Observations du jury s'affichent correctement

[pix-bot-github]

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

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• 1D
• Admin
• API
• Audit Logger

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

• scalingo front
• scalingo api
• scalingo audit-logger

[AndreiaPena]

Testé func :

• Export des résultats d'une classe

  • Sur orga.org en ?lang=en : ✅

• Sur orga.org en ?lang=fr : messages en français ✅

• Sur orga.fr : ✅

• Si pas de commentaires, ne rien afficher : ✅

• Commentaires jury sur Admin :

  • admin.fr : J'ai obtenu les commentaire en anglais parce que Pix Admin est la seule app il me semble à ne pas forcer le français quand on est sur du .fr (à vérifier). L'app se base du coup sur les langues de l'utilisateur du navigateur, et pour ma part j'avais mis anglais en langue prio. D'où ce résultat.

Du coup en BDD j'ai ça

Et on me restitue uniquement les commentaires auto liés à la fraud parce que FRAUD en commentByAutoJury ✅

autrement :

• Résultat pour le candidat sur Pix App :
  • gestion des trad en français et anglais ✅
  • j'ai tenté une clé bidon sur commentByAutoJury pour voir et ça met le commentaire classique se trouvant dans commentForCandidate ✅

[cberthou]

D'un point de vue métier, je suis étonné que le commentaire automatique prenne le pas sur le commentaire manuel, mais j'imagine que ça a été validé. Je suis un peu étonné de l'utilisation du terme fallbackComment qui semblerait mieux nommé en manualJuryComment, mais peut-être que j'ai mal compris le métier.

[Osirisxxl]

D'un point de vue métier, je suis étonné que le commentaire automatique prenne le pas sur le commentaire manuel, mais j'imagine que ça a été validé. Je suis un peu étonné de l'utilisation du terme fallbackComment qui semblerait mieux nommé en manualJuryComment, mais peut-être que j'ai mal compris le métier.

@cberthou Le fallback est demandé comme ça en effet. Dans les tickets à venir, on enlève la possibilité de mettre un commentaire manuel aussi. Le fallback c'est plus pour l'historique donc. manualJuryComment ça colle aussi en effet. Je pense que fallback c'est le premier nom qu'on a sorti et qu'on l'a jamais remis en question 😞.