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] Test blockly to generate quests #11520

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

[TECH] Test blockly to generate quests #11520

wants to merge 6 commits into from

Conversation

frinyvonnick
Copy link
Member

@frinyvonnick frinyvonnick commented Feb 26, 2025

🥞 Problème

C'est assez fastidieux d'écrire les quêtes pour les POs et elles peuvent devenir rapidement très longue comme celle des attestations pour les 6ème. Le format JSON peut être source de typos qu'il devient difficile de retrouver quand la quête fait plusieurs centaines de lignes.

🥓 Proposition

Ajouter un éditeur de quête dans PixAdmin qui utilise un modalité d'expression via des blocs. Cela semble particulièrement adapté aux quêtes donc la profondeur (blocs nestés) peut être importante.

🧃 Remarques

Etant un petit outil à part fait en après-midi tech, il n'y a pas spécialement de tests automatiques qui ont été fait. D'autant que ça reviendrait en partie à tester la librairie. Le risque par contre est que l'outil soit cassé par une montée de version sans qu'on s'en rend compte. Cependant, en l'état cela apporte déjà de la valeur en interne.

😋 Pour tester

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

@frinyvonnick frinyvonnick added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Feb 27, 2025
@frinyvonnick frinyvonnick marked this pull request as ready for review February 27, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed team-noa(h) noa - noah 🎾
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants