ace-lab / ace-lab/pl-faded-parsons

Validate info.json Configuration Early

Aperta
#55 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement good first issue pl-element
Lingua principale
Python
Stelle
1
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

As it stands, the system only checks for the resources it needs to render, which means that the system will sometimes err only when Save & Grade is hit due to ./solution being missing or showCorrectAnswer being un-set.

It is worth validating the full JSON config before rendering to keep the "once a component appears to work, it works" philosophy in place like the rest of PL.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Start at the code path that handles render and Save & Grade and currently checks only required resources. Locate where `info.json` is loaded and introduce/extend pre-render validation so missing fields (for example `showCorrectAnswer`) and required resources like `./solution` are rejected before rendering. Then run the relevant render/Save & Grade tests or checks and confirm invalid configs now fail early with a clear error.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
backend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
43/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.