Comments (1)
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
yonBlur
(con elon
ya se sobreentiende que es un evento. ParainputStyle
lo cambiaría aclassName
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.
Related Issues (6)
- Wrong patterns HOT 5
- Cedula panama en PHP HOT 1
- Validación de identificadores RUC HOT 1
- Python Implementation HOT 2
- El patron no funciona en input de html HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cedula-panama.