"required": true for a property. Does the codgen support it?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- devtools
Línea de trabajo
Start by tracing how the generator reads Schema Object properties and determines whether each TypeScript property is optional; no source file or test is named in the issue. Compare handling of the object-level required array with property-level required values, then add or update coverage so the supported behavior is explicit in the generated output.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I have a Schema Object
{
"Obj": {
"description": "Obj",
"required": [
"id",
"foo"
],
"properties": {
"id": {
"type": "string",
"pattern": "^[0-9a-f]{32}$"
},
"foo": {
"type": "string",
"pattern": "^[0-9a-f]{32}$"
},
"bar": {
"type": "string",
"pattern": "^[0-9a-f]{32}$"
}
}
}
}
With this ShemaObject, a type generates perfectly fine.
/**
* Obj
*/
export type Obj = {
/**
* @pattern ^[0-9a-f]{32}$
*/
id: string;
/**
* @pattern ^[0-9a-f]{32}$
*/
foo: string;
/**
* @pattern ^[0-9a-f]{32}$
*/
bar?: string;
};
But if I move/distribute required to properties like this
{
"Obj": {
"description": "Obj",
"properties": {
"id": {
"type": "string",
"pattern": "^[0-9a-f]{32}$",
"required": true
},
"foo": {
"type": "string",
"pattern": "^[0-9a-f]{32}$",
"required": true
},
"bar": {
"type": "string",
"pattern": "^[0-9a-f]{32}$"
}
}
}
}
It generates this:
/**
* Obj
*/
export type Obj = {
/**
* @pattern ^[0-9a-f]{32}$
*/
id?: string;
/**
* @pattern ^[0-9a-f]{32}$
*/
foo?: string;
/**
* @pattern ^[0-9a-f]{32}$
*/
bar?: string;
};
Conclusion:
It ignores this approach of requiring fields if the required is set at the property level. Is it a bug? Is it possible to set up this approach?
- Lenguaje dominante
- TypeScript
- Estrellas
- 634
- Forks
- 83
- 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 fabien0102/openapi-codegen
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
fabien0102/openapi-codegen#343 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
fabien0102/openapi-codegen#342 ·
-
deepMerge mutates its arguments, corrupting request payloads across callsPosiblemente ocupada @richard-willis-chevin la tomó hace 43 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
fabien0102/openapi-codegen#349 ·
-
Error when migrating to typesrcipt v7Posiblemente ocupada @fujikky la tomó hace 43 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
fabien0102/openapi-codegen#348 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
fabien0102/openapi-codegen#344 ·
Todos los issues de fabien0102/openapi-codegen
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nats-io/nats.docs.v2#101 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
profullstack/ugig.net#601 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
github/codeql-action#4202 ·
Los mantenedores suelen responder en 1 día
-
agents: formatReport/reportOrigin only importable through an entry that loads every runtime (~1.5 s)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día