updated react to 16.1.1, removed linkedInputs #1082
Conversation
chrisdugne
commented
Nov 14, 2017
•
chrisdugne
commented
- react 16
- updated enzyme
chrisdugne
force-pushed
the
react-16
branch
2 times, most recently
from
98bb6c9 to
f35ce74
Compare
Codecov Report
@@ Coverage Diff @@
## master #1082 +/- ##
=======================================
Coverage 95.15% 95.15%
=======================================
Files 233 233
Lines 3489 3489
=======================================
Hits 3320 3320
Misses 169 169
Continue to review full report at Codecov.
|
| @@ -55,7 +54,7 @@ const ButtonContent = props => { | |||
|
|
|||
| default: | |||
| return ( | |||
| <LinkedInput | |||
| <input | |||
There was a problem hiding this comment.
Ça servait à quoi d'utiliser LinkedInput et pourquoi on (peut) s'en débarasse(r) maintenant ?
There was a problem hiding this comment.
c'etait deja deprecié pour react 15 mui/material-ui#2880 (comment)
There was a problem hiding this comment.
cela dit, @godu l'a utilisé pour le input-text de recherche des users dans setup principalement: de ce que j'ai compris cest pour mettre en en file les renders à chaque onChange de chaque lettre.
A priori on ne devrait pas avoir ce pb avec react 16, il va falloir tester je n'ai pas assez de users en local
There was a problem hiding this comment.
pour les autres inputs que jai testé ca fonctionne tres bien.
| <div | ||
| data-name="cover" | ||
| className={style.ctaWrapper} | ||
| onClick={!disabled ? onClick : noop} |
There was a problem hiding this comment.
disabled ? noop : onClick plutôt, moins de gymnastique mentale à faire
(idem plus bas)
* updated react to 16.1.1, removed linkedInputs * updated enzyme * updated yarn.lock * migrated enzyme to use adapter for react 16 * fixed few components
