Nested form data #254
Open
Nested form data #254
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Remove `questionpy_sdk.webserver.controllers.question._form_data`. Ref: #249
- Consistently use `path` to reference form elements instead of string key (`name`). - Discard custom model for `file_upload`, `repeat`, and `rich_text_editor` elements. - Extract `<ValidationFeedback>` component. - `useCommon` -> `useId`, and `usePath`. Closes #249
- Always add validation state to `<FormGroup>` and `<ValidationFeedback>` to all form elements. - Replace manual margins with Bootstrap's `vstack` for vertical spacing. - Restyle repetitions using card layout. - Correct label association for select elements.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Das "over-the-wire"-Format für die Options-Form-Daten ist nun nicht mehr flach, sondern geschachtelt. So funktioniert es ohne weitere Umformungen direkt für Worker sowie Web-Applikation.
Siehe #249
81ba838
Die größten Änderungen im Backend
questionpy_sdk.webserver.controllers.question._form_dataist überflüssig und wurde entfernt, undOptionsFormData-Typ hat sich geändert.9bd97a3
Die größten Änderungen:
ElementPath. Vorher gab es zwei Arten: den Pfad in der verschachtelten Ansicht, sowiename, der Key in der flachen Ansicht.namewurde komplett entfernt und alle Stellen nutzen nur noch den Pfad.repeat,file_upload,rich_text_editor) ist obsolet. Alle Form-Elemente nutzen einfachuseModel()bzw. diegetValue()-,setValue()-Funktionen direkt.Außerdem noch
<ValidationFeedback>extrahiert unduseCommoninusePathunduseIdaufgeteilt, da ich die Benennung klarer finde, vor allem da wir keinnamemehr haben.a83b21c
Es haben sich noch ein paar Refaktorisierungen ergeben, bzw. mir sind bei der Überarbeitung noch ein paar Sachen aufgefallen, die ich versucht habe der Übersicht halber in einem einzelnen Commit zu isolieren.