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

Question UI: erlaubte Rollen mittels Regex #100

Merged
merged 1 commit into from
Jun 28, 2024
Merged

Conversation

MartinGauk
Copy link
Contributor

Die Methode _hide_if_role verhielt sich nicht kompatibel zum Moodle-Plugin und unserer Definition im internen Wiki. Mehrere Rollen in qpy:if-role können mittels | oder mit beliebig vielen Whitespace Characters getrennt angegeben werden. Bei der Gelegenheit habe ich noch etwas aufgeräumt und die context Option durch roles ausgetauscht.

Dieser PR hängt von #99 ab.

@MartinGauk MartinGauk requested a review from tumidi June 10, 2024 15:29
Copy link
Contributor

@tumidi tumidi left a comment

Choose a reason for hiding this comment

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

Ich denke, ein Enum würde sich anbieten die Rollen zu modellieren (siehe Kommentare im Code).

Dann bekäme man:

QuestionDisplayOptions().model_dump_json(indent=2)
{
  "general_feedback": true,
  "feedback": true,
  "right_answer": true,
  "roles": [
    "TEACHER",
    "SCORER",
    "PROCTOR",
    "DEVELOPER"
  ],
  "readonly": false
}

anstatt

{
  "general_feedback": true,
  "feedback": true,
  "right_answer": true,
  "roles": {
    "teacher": true,
    "developer": true,
    "scorer": true,
    "proctor": true
  },
  "readonly": false
}

questionpy_sdk/webserver/question_ui.py Outdated Show resolved Hide resolved
questionpy_sdk/webserver/question_ui.py Outdated Show resolved Hide resolved
tests/webserver/test_question_ui.py Outdated Show resolved Hide resolved
@MartinGauk MartinGauk force-pushed the hide-if-role-split branch from dfe4657 to 47edf4a Compare June 14, 2024 15:12
@MartinGauk
Copy link
Contributor Author

Danke dir, ich habe alles übernommen. Ich habe noch geändert, dass die angegebenen Rollen im XML in großer Schreibweise abgeglichen werden, um mit der Großschreibung im enum kompatibel zu sein. Das hat auch den Vorteil, dass die Rollen akzeptiert werden, egal ob sie groß- oder kleingeschrieben sind, was ja sinnvoll ist. Ich habe das auch in Tefen dokumentiert.

Base automatically changed from refactor_question_ui to dev June 17, 2024 09:07
@MartinGauk MartinGauk force-pushed the hide-if-role-split branch from 47edf4a to 7562f3c Compare June 28, 2024 16:24
@MartinGauk MartinGauk merged commit 5d3e97d into dev Jun 28, 2024
6 checks passed
@MartinGauk MartinGauk deleted the hide-if-role-split branch June 28, 2024 16:26
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.

2 participants