-
Notifications
You must be signed in to change notification settings - Fork 56
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] Bloquer la finalisation de session selon certaines conditions sur Pix Certif (PIX-14717). #10473
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 : |
5a418d5
to
7f3c060
Compare
certif/app/controllers/authenticated/sessions/details/parameters.js
Outdated
Show resolved
Hide resolved
493cf29
to
afd7f38
Compare
afd7f38
to
94a5d68
Compare
…is v3 and a certification candidate has complementary subscription (only one)
94a5d68
to
d84fd08
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.
Je publie quand meme mes commentaires malgre le merge de la PR entre le debut et la fin de ma revue de la PR
const hasCoreSubscription = this.subscriptions.some((sub) => sub.isCore); | ||
const hasComplementarySubscription = this.subscriptions.some((sub) => sub.isComplementary); | ||
return !hasCoreSubscription && hasComplementarySubscription; |
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 de simplification
return this.subscriptions.every((sub) => sub.isComplementary);
{{t "common.actions.update"}} | ||
</PixButtonLink> | ||
{{#if this.sessionManagement.isFinalized}} | ||
{{#if this.isSessionFinalizationTemporarilyBlocked}} |
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.
Cette suite d'imbrications et d'enchainements logiques au niveau templating me fait dire que l'on devrait aller vers de la separation de composants / refacto des conditions
{{t "pages.sessions.detail.parameters.actions.finalizing"}} | ||
</PixButtonLink> | ||
{{#if this.sessionManagement.isFinalized}} | ||
<p class="session-details-row__session-finalized-warning"> |
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.
Les differentes class
sont pas forcement coherentes, et pas super BEM. L'historique du fichier aide certes pas mais ca renforce ma suggestion de decoupage ci-dessus :)
🍂 Problème
Dans le cadre du sujet “Compatibilité Pix v3/Pix+”, un candidat doit pouvoir passer un test de certification Pix+ seul. Néanmoins l’implémentation du scoring du Pix+ seul a été décalé dans le temps et la finalisation de session dans ce cas de figure ne peut être réalisée.
🌰 Proposition
Ne pas afficher le bouton de finalisation de session dans le cas suivant :
🎃 Remarques
Dans le premier commit, je fait un mini refacto en supprimant un bouton de modification de session.
Une autre PR est prévue pour la partie API
🪵 Pour tester
Vérifier le détail d'une session V2 (Non régression)
Vérifier le détail d'une session V3 (sans certif complémentaire) (Non régression)
Vérifier le détail d'une session V3 (avec Pix+ seul)