Pre parsing validation
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
- Área
- backend
Línea de trabajo
Empieza leyendo el issue #166 y rastrea dónde se analizan y validan los campos de formulario; este issue no menciona archivos ni tests. Compara los enfoques propuestos de is module y node-validator, y después aclara el comportamiento previsto para los objetos enviados a campos de tipo string. Se considera terminado cuando el enfoque de validación y el comportamiento compatible estén acordados, con cobertura para el caso de conversión reportado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I've read the following issue: #166 and I'm wondering how we can enforce type validation (using is module or node-validator).
If you submit an object to a string field, it is viewed as valid ("[Object object]"). Of course, I don't think it is the desired behaviour. What do you think about it?
Or maybe providing a generic field and we add this kind of validation:
exports.typeValidator = function (type, message) {
if(!is.fn(is[type])) {
throw new TypeError('this type is not defined');
}
var msg = message || 'Please enter a valid value';
return function (form, field, callback) {
if (is[type](field.data)) {
callback();
} else {
callback(msg);
}
};
};
- Lenguaje dominante
- JavaScript
- Estrellas
- 1k
- Forks
- 160
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de caolan/forms
-
Feature Requests Needs More Info
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
Cut a new release?Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
Feature Requests pull request wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
-
simple checkboxAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
Todos los issues de caolan/forms
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
SACGF/variantgrid#2341 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Unsampled pageload span is profiled when profileLifecycle is 'trace' (browser UI profiling)Posiblemente ocupada @Jxxunnn la tomó hoy. AbiertoBrowser Waiting for: Product Owner
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
getsentry/sentry-javascript#25497 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[External Plugin]: rostrumAbiertoexternal-plugin needs-review:HIGH ready-for-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
github/awesome-copilot#5162 · 3 comentarios ·
Los mantenedores suelen responder en 1 día