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

#140: generated_id #144

Merged
merged 4 commits into from
Jan 8, 2025
Merged

#140: generated_id #144

merged 4 commits into from
Jan 8, 2025

Conversation

MHajoha
Copy link
Member

@MHajoha MHajoha commented Dec 19, 2024

Ich habe mich für eine Repetition-unspezifische Lösung entschieden, das schien mir einfacher, als jedem Repetition-Item eine ID zuzuweisen.

Closes #140.

Server-PR: questionpy-org/questionpy-server#126
Plugin-PR: questionpy-org/moodle-qtype_questionpy#150

Und damit schöne Feiertage und einen guten Rutsch!

@MartinGauk
Copy link
Contributor

Mir würde ein RepeatedElementsBaseModel gefallen, das bereits id: str = generated_id() beinhaltet. repeat könnte trotzdem beliebige FormModel akzeptieren, aber in den meisten Fällen sollte man von RepeatedElementsBaseModel erben, da die ID sehr sinnvoll ist.

Das neue Beispiel sollte eigentlich eher singlechoice heißen!?

@MHajoha
Copy link
Member Author

MHajoha commented Jan 7, 2025

Mir würde ein RepeatedElementsBaseModel gefallen, das bereits id: str = generated_id() beinhaltet. repeat könnte trotzdem beliebige FormModel akzeptieren, aber in den meisten Fällen sollte man von RepeatedElementsBaseModel erben, da die ID sehr sinnvoll ist.

Naja, in beiden Fällen ist es opt-in, also eher eine stilistische Entscheidung. Da macht eine eigene Superklasse es mMn nur undurchsichtiger. Und weniger zu Tippen ist es auch kaum.

Das neue Beispiel sollte eigentlich eher singlechoice heißen!?

Stimmt natürlich, ich wusste beim Erstellen noch nicht so ganz, welches von beiden es wird 🙈

@MHajoha MHajoha merged commit 2f51d6a into dev Jan 8, 2025
6 checks passed
@MHajoha MHajoha deleted the repetition-id branch January 8, 2025 14:17
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.

UUID in Repeating Elements hinterlegen
2 participants