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

[TECH] Passer la page de détails d'une session en .gjs (PIX-13251) #9426

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

mcampourcy
Copy link
Contributor

@mcampourcy mcampourcy commented Jul 2, 2024

🦄 Problème

Le format de composant utilisant la balise template va devenir le format par défaut d'ici la fin de l'édition Polaris de Ember.
Tous nos composants sont encore au format hbs. On peut commencer à les passer au format .gjs

🤖 Proposition

Passer les composants de la page d’affichage des détails d’une session en gjs.

💯 Pour tester

NON REG

  • Se connecter sur Pix Certif avec certif-pro@example.net, aller dans la liste des sessions, cliquer sur une session finalisée, puis onglet "Détails"
  • Si aucun candidat n'est Cléa, créer une session avec un candidat Cléa et la finaliser
  • Vérifier que l'onglet a le bon titre : Détails | Session {sessionId} | Pix Certif
  • Vérifier que l’affichage est correct (cf capture d'écran)
  • Vérifier que "Revenir à la liste des sessions" renvoie à la liste des sessions
  • Vérifier que le numéro de session est indiqué dans le titre
  • Vérifier que les dates et heure sont OK
  • Vérifier que les boutons pour télécharger PV, feuille d'émargement et kit surveillant fonctionnent toujours (pas besoin de vérifier le fichier)
  • Sur l'encart Cléa, vérifier que le bouton de téléchargement fonctionne toujours, et que le lien renvoie vers la bonne page

@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 :

@mcampourcy mcampourcy force-pushed the pix-13251-page-details-from-hbs-to-gjs branch 4 times, most recently from 260c90b to 9e2f233 Compare July 4, 2024 08:54
@mcampourcy mcampourcy marked this pull request as ready for review July 4, 2024 09:45
@mcampourcy mcampourcy force-pushed the pix-13251-page-details-from-hbs-to-gjs branch 2 times, most recently from ebe43a0 to 8b8b76b Compare July 4, 2024 13:32
@mcampourcy mcampourcy marked this pull request as draft July 4, 2024 19:41
@mcampourcy mcampourcy force-pushed the pix-13251-page-details-from-hbs-to-gjs branch from 8b8b76b to 908109f Compare July 5, 2024 08:32
@mcampourcy mcampourcy marked this pull request as ready for review July 5, 2024 08:50
@AndreiaPena
Copy link
Member

AndreiaPena commented Jul 5, 2024

HS à la PR : 🥲 il manque des icônes AWARD/LINK (cause: la montée de version de Pix UI)
Possible de les ajouter dans la PR ? 😅

Capture d’écran 2024-07-05 à 14 58 36

@AndreiaPena AndreiaPena added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Jul 5, 2024
@@ -1,6 +1,7 @@
module.exports = function () {
return {
'free-solid-svg-icons': [
'award',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13251-page-details-from-hbs-to-gjs branch from e3490c8 to 7d79766 Compare July 8, 2024 08:21
@pix-service-auto-merge pix-service-auto-merge merged commit 50e07dc into dev Jul 8, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-13251-page-details-from-hbs-to-gjs branch July 8, 2024 08:27
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