search

InseeFr / Lunatic

Library of questionnaire components
https://inseefr.github.io/Lunatic/
MIT License
20 stars 21 forks source link

Refactor handleChange => handleChanges #950

Closed Grafikart closed 4 months ago

Grafikart commented 4 months ago

Problème

Certains composants changent plusieurs valeurs lors de certaines opérations (suggester, pairwise). Dans la version actuelle la fonction handleChange est appelé plusieurs fois en déclenchant un nouveau rendu à chaque fois.

Solution

On retire la méthodehandleChange pour une méthode handleChanges qui va recevoir un tableau de réponses. Ce changement n'affecte pas les composants personnalisés.

Remarque

En faisant la modification je remarque que les composants personnalisés ont accès à la propriété handleChange en plus de la méthode personnalisée onChange. Je serais tenté de retirer ce passage de props car je pense que les composants personnalisés devraient avoir accès à cette méthode.

sonarcloud[bot] commented 4 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud