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] Ajout d'options dans le challenge pour Pix Junior #869

Conversation

aurelie-crouillebois
Copy link
Contributor

🥞 Problème

Dans Pix Junior, les données focus et timer sont détournées de leur usage pour gérer des cas spécifiques de validation des épreuves :

  • les leçons contenant uniquement des vidéos ou des fiches récapitulatives sans besoin de validation
  • les épreuves embeds avec validation interne comme les gdevelop ou les embeds pixwriter de mise en forme.

🥓 Proposition

Ajouter 2 nouvelles boites à cocher spécifiques dans le challenge.

  • "Sans validation (Pix Junior)" (noValidationNeeded)au dessous du type d'épreuve pour spécifier qu'aucune étape de vérification n'est attendue pour ce "challenge"
  • "Avec validation interne (Pix Junior)" (hasEmbedInternalValidation) au niveau des embeds pour indiquer que l'embed gère lui même ses règles de validation.

🧃 Remarques

Ces données sont modifiables sur un prototype uniquement.

😋 Pour tester

Ouvrir l'écran de création d'un nouveau challenge :
-> les boites à cocher sont présentes et non cochées
Cocher les cases, et enregistrer le challenge.
-> les boites apparaissent bien comme cochées dans la vue de consultation
Ajouter une déclinaison.
-> les boites à cocher sont absente de l'édition de la déclinaison.
Générer une release.
-> Les 2 challenges (prototype et déclinaison) ont les valeurs noValidationNeeded et hasEmbedInternalValidation à true.
Modifier le prototype. Décocher l'une des cases et générer une release.
-> Les 2 challenges (prototype et déclinaison) ont la valeur correspondant à la case décochée à false.

@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://pix-lcms-review-pr869.osc-fr1.scalingo.io
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-lcms-review-pr869/environment

@rouxxi rouxxi force-pushed the pix-16177-add-lesson-and-auto-validation-attributes-to-challenge branch from a124c6b to 328adb6 Compare January 30, 2025 08:57
@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-16177-add-lesson-and-auto-validation-attributes-to-challenge branch from 328adb6 to 0d5dffb Compare January 30, 2025 14:09
@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-16177-add-lesson-and-auto-validation-attributes-to-challenge branch from 0d5dffb to 75ba1da Compare January 30, 2025 14:17
Copy link
Member

@laura-bergoens laura-bergoens left a comment

Choose a reason for hiding this comment

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

Du coup le script pour les données vous le faites dans une PR consécutive ?

aurelie-crouillebois and others added 11 commits January 31, 2025 13:30
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
…outes

Co-authored-by: Gwenvaël Laskar <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
…the challenge model

These options can be modified on prototype only.
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16177-add-lesson-and-auto-validation-attributes-to-challenge branch from 75ba1da to 98c9365 Compare January 31, 2025 13:30
@pix-service-auto-merge pix-service-auto-merge merged commit 27433ac into dev Jan 31, 2025
3 of 6 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16177-add-lesson-and-auto-validation-attributes-to-challenge branch January 31, 2025 13:36
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