Wrapper para React about cedula-panama HOT 1 OPEN

Está muy chévere! Estuve mirando el código y está súper neat. Además, no conocía nwb, y tiene buena pinta. Cuando he tenido que crear una biblioteca de componentes react siempre ha sido un dolor configurar los builders...

Algunas sugerencias para tu consideración:

• Para nombres los eventos como propTypes, estaría a favor de usar:onChange y onBlur (con el on ya se sobreentiende que es un evento. Para inputStyle lo cambiaría a className y labelClassName. Al tener Style en el nombre asumo que tengo que meter directamente los css no un nombre de clase que es lo que me ha parecido entender que es lo que se espera.

• ¿Cómo se establece el valor inicial? Por ejemplo, en un formulario de actualización donde ya tengo una cédula en la base de datos.

• Si necesito pasar otros parametros al input como un data-* creo que no se podría. Lo puedes solucionar con algo así:

const CedulaPanamaField = ({ propiedad1, propiedad 2, ...others}) => { 

   return(
      <input  propiedades controladas  {...others} />
 }

• Si agregas documentación en las proptypes en VSCode te aparece la info sin tener que abrir el repo. Muy útil durante desarrollo

/** Esta es la doc de una propiedad */

• Me ha parecido entender que los eventos solo devuelven un bool. Creo que podría estar bien que el objeto resultante de la validación (validate()) estuviera disponible como segunda propiedad de los handlers. Esto es útil porque el objeto te da por ejemplo el campo dividido (si en la bbdd lo tienes así) y también te permite ser un poco más fino en los mensajes de error.

Por último, quizás estaría bien para facilitar el descubrimiento a otros desarrolladores tener todo lo referente a la cédula (o incluso helpers muy específicos de Panamá) en un único repo o varios repos en una organización. ¿Qué te parece?

Saludos!

from cedula-panama.