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] Ajout de nouvelles colonnes pour les challenges pix-junior #11259

Merged

Conversation

aurelie-crouillebois
Copy link
Contributor

@aurelie-crouillebois aurelie-crouillebois commented Jan 29, 2025

🥞 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 colonnes dans le challenge permettant de stocker ces informations.

😋 Pour tester

Non régression :

  • Faire un npm run cache:refresh - l'opération est ok et les applications pix-app et pix-junior fonctionnent toujours
  • Faire un npm run db:seed - l'opération est ok et les applications pix-app et pix-junior fonctionnent toujours

Lancer la commande npm run db:migration :
-> les colonnes noValidationNeeded et hasEmbedInternalValidation sont ajoutées à la table learningcontent.challenges (booléen avec valeur false par défaut)
Lancer la commande npm run db:rollback:latest 2 fois.
-> les colonnes ont bien disparu

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

@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-16314-add-pix-junior-columns-to-challenges branch from aa88f91 to 13919b7 Compare January 29, 2025 09:43
Copy link
Contributor

@mcampourcy mcampourcy left a comment

Choose a reason for hiding this comment

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

Tech review OK

@rouxxi rouxxi added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Jan 31, 2025
@mcampourcy
Copy link
Contributor

Func review OK

@rouxxi rouxxi force-pushed the pix-16314-add-pix-junior-columns-to-challenges branch from 9abf04d to 83c0788 Compare January 31, 2025 15:23
@pix-service-auto-merge pix-service-auto-merge merged commit 49ab8b8 into dev Jan 31, 2025
6 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16314-add-pix-junior-columns-to-challenges branch January 31, 2025 15:35
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