jaredpalmer/formik

Array-helper `insert` adds value to error state

Ouverte

#1 092 ouverte le 8 nov. 2018

Voir sur GitHubGuide contributeur
 (5 commentaires) (3 réactions) (0 personne assignée)TypeScript (2 801 forks)batch import
Priority: MediumType: Bughelp wantedpinned

Métriques du dépôt

Stars
 (34 255 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

🐛 Bug report

Current Behavior

When i add an item to a FieldArray using insert the value is also inserted into the errors state (see example below).

Expected behavior

Only insert the value to the values state, not to the errors state.

Reproducible example

https://codesandbox.io/s/v87n4p6023

Your environment

Software Version(s)
Formik 1.3.1
React 16.5.2

Guide contributeur

Direction de recherche
Le bug est que la méthode `insert` dans FieldArray de Formik ajoute incorrectement la valeur à l'état des erreurs. Commencez par examiner la fonction `insert` dans `packages/formik/src/FieldArray.tsx`. Regardez comment elle met à jour à la fois les valeurs et les erreurs ; elle utilise probablement `setIn` ou des aides similaires. La correction doit garantir que `insert` ne met à jour que les valeurs, pas les erreurs. Le CodeSandbox fourni (https://codesandbox.io/s/v87n4p6023) reproduit le problème. Vérifiez les commentaires existants pour toute piste supplémentaire ou tentative de correction.
Stack technique
reacttypescript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactFormik
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75