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

JavaScript-Aufrufe in QPPE und Datenstrukturen #120

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

MartinGauk
Copy link
Contributor

questionpy_common/api/attempt.py Outdated Show resolved Hide resolved
questionpy_common/api/attempt.py Show resolved Hide resolved
@larsbonczek
Copy link

Was ist eigentlich die Überlegung dahinter, Parameter für die JS Funktion nur in Form eines Objektes zu übergeben (und nicht in einem Array)?

@MartinGauk
Copy link
Contributor Author

Was meinst du genau? Durch das JSON ist man ja schon sehr flexibel und gerne kann man auch direkt ein Array übergeben. Mittels Destructuring Assignments kann man die Daten leicht auspacken, wenn man will.

@larsbonczek
Copy link

Ich habe nur gefragt, weil es in Moodle ja anders ist. Aber mit den Destructuring Assignments sollte es klappen, das stimmt.

questionpy_common/api/attempt.py Outdated Show resolved Hide resolved
@MartinGauk
Copy link
Contributor Author

Ich habe den Commit neu gemacht. Es gibt zu vorher noch eine wesentliche Änderung: Die Attempt/Options Endpoints geben nun auch noch zurück, welche Pakete bei der Ausführung involviert sind (package_dependencies mit den Namen und den Hashes). Die Information wird vom Moodle-Plugin benötigt, um die Referenzen zu den JS-Dateien auflösen zu können, d.h. um die Importmap bilden zu können (das Plugin kennt ggf. nur den Hash des Pakets, aber nicht den Namespace/Shortname).
Die Information zu den Abhängigkeiten werden wir auch noch brauchen, wenn wir irgendwann dynamische Paketabhängigkeiten mal haben sollten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants