jaredpalmer/formik

setFieldError does not prevent handleSubmit

Open

#1.111 geöffnet am 15. Nov. 2018

Auf GitHub ansehen
 (10 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)TypeScript (2.801 Forks)batch import
Priority: MediumType: Bughelp wantedpinned

Repository-Metriken

Stars
 (34.255 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

🐛 Bug report

This looks like it may be a possible regression on #317, as I can replicate it in the exact same way.

Current Behavior

After setting an error using setFieldError, the handleSubmit function is successfully fired.

Expected behavior

The form should not submit successfully, as there is an error.

Reproducible example

Click "Set Error". You will then be able to successfully submit. https://codesandbox.io/s/43owlv689

Suggested solution(s)

Your environment

Software Version(s)
Formik 1.3.2
React 16.4.2
TypeScript
Browser Chrome
npm/Yarn 6.1.0
Operating System OSX

Contributor Guide

Research-Richtung
Der Bug ist, dass `setFieldError` die FormularĂŒbermittlung nicht verhindert. Sieh dir den verlinkten PR #317 an, der `setFieldError` eingefĂŒhrt hat, und untersuche, wie `handleSubmit` mit dem Fehlerstatus interagiert. Das reproduzierbare Beispiel im Issue (CodeSandbox) kann verwendet werden, um das Problem nachzuvollziehen. ÜberprĂŒfe den aktuellen Formik Quellcode, insbesondere die Funktionen `handleSubmit` und `validateField`, um das erwartete Verhalten zu verstehen. ErwĂ€ge, vor dem Aufruf von `onSubmit` eine FehlerprĂŒfung hinzuzufĂŒgen.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschĂ€tzte Implementierungsschwierigkeit fĂŒr neue Contributors, von 1 fĂŒr sehr kleine Änderungen bis 5 fĂŒr Expertenarbeit.
3
GeschÀtzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitĂ€tsstatusWie verfĂŒgbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nĂ€chsten Schritt erklĂ€rt.
Klar
Voraussetzungen
ReactFormik
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schĂ€tzt, wie zugĂ€nglich dieses Issue fĂŒr First-Time Contributors ist.
70