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] Ajouter un sous-composant d'affichage des alertes utilisateur (PIX-16722). #11813

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Mar 24, 2025

🌸 Problème

Dans l'objectif d'alléger le template admin/app/templates/authenticated/certifications/certification/informations.hbs, il est nécessaire de réussir à alléger son unique composant admin/app/components/certifications/certification/informations.gjs.

🌳 Proposition

Commencer à faire de la composition en créant un sous-composant qui gérera l'affichage de la partie "Issue reports".

🤧 Pour tester

En RA, vérifier que dans le détail d'une certification, le bloc s'affiche bien lorsqu'un issue-report existe.
S'il n'y en a pas, le bloc n'est pas censé être affiché.

@Jeyffrey Jeyffrey self-assigned this Mar 24, 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 :

@Jeyffrey Jeyffrey force-pushed the pix-16722-improve-certif-issue-reports-sub-component branch from b1a75db to 2d653bf Compare March 24, 2025 16:45
Copy link
Member

@yaf yaf left a comment

Choose a reason for hiding this comment

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

🚀

@Jeyffrey Jeyffrey force-pushed the pix-16722-improve-certif-issue-reports-sub-component branch from 2d653bf to 7080521 Compare March 25, 2025 08:44
@alexandrecoin
Copy link
Contributor

Capture d’écran 2025-03-25 à 11 04 42

@Jeyffrey Jeyffrey added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Mar 25, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16722-improve-certif-issue-reports-sub-component branch from 7080521 to 0de2d4a Compare March 25, 2025 13:05
@pix-service-auto-merge pix-service-auto-merge merged commit 7361e4f into dev Mar 25, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16722-improve-certif-issue-reports-sub-component branch March 25, 2025 13:11
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.

5 participants