Add Formik Example to React Sample App #5414
Conversation
…m:public-ui/kolibri into 5371-scenario-komplexes-workflow-formular * '5371-scenario-komplexes-workflow-formular' of github.com:public-ui/kolibri: fix: tests chore: better rules docs: update js docs Revert "docs: optimize jsdoc"
…-formular' into 5371-scenario-komplexes-workflow-formular * origin/5371-scenario-komplexes-workflow-formular: Add async validation for individual appointments
| _label: '1. Einwohnermeldeamt wählen', | ||
| }, | ||
| { | ||
| _label: '2. Freie Termine', |
There was a problem hiding this comment.
Ich würde hier gerne jeweils noch eine _disabled-Property hinzufügen, wenn die vorherige Sektion nicht valide ist. Mir ist bloß noch nicht klar, wie wir das mit Formik erreichen können, weil auf dieser Ebene keinen Zugriff auf die Form-Instanz bzw. die Values haben.
Ich lasse den Punkt erst einmal offen und komme ggf. später darauf zurück.
| (times) => { | ||
| if (!ignoreResponse) { | ||
| setAvailableTimes(times); | ||
| void form.setFieldValue('time', times[0].value); |
There was a problem hiding this comment.
Hier war ich praktisch gezwungen, einen Wert vorauszuwählen: KoliBri markiert per default den ersten radio button als checked, auch wenn der value mit keiner der Optionen übereinstimmt (z.B. leerer value). Das ist vermutlich ein Bug, der in KoliBri behoben werden sollte, weil so keine "leere Auswahl" von Radio-Buttons möglich ist. Getestet mit '', undefined und null.
| id="field-date" | ||
| _label="Datum" | ||
| _value={field.value} | ||
| _error={form.errors.date || ''} |
There was a problem hiding this comment.
|| '') ist unbedingt notwendig, weil wenn der error von einem String zurück zu undefined geändert wird, dies nicht im internen State übernommen wird. Der Fehler bleibt also immer sichtbar.
Ich habe hier einiges ausprobiert, aber keine Lösung gefunden, die nicht gleichzeitig ein Breaking Change wäre. Ich würde vorschlagen, dass wir uns das Thema einmal zusammen anschauen.
Relevante Stelle: packages/components/src/utils/prop.validators.ts:102
sdvg
changed the title
…371-scenario-komplexes-workflow-formular
deleonio
force-pushed
the
5371-scenario-komplexes-workflow-formular
branch
from
42e094d to
86a37fb
Compare
No description provided.