Correct handling of `style` and `explode`.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- tooling
Línea de trabajo
Start by locating the query-parameter generation entry point and its handling of OpenAPI parameter style and explode defaults. Compare the generated TypeScript with the OpenAPI 3.0 Parameter Object rules, then add or update tests so the intended output for form-style query parameters is explicit and covered.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
OpenAPI spec:
targets:
get:
tags:
- targets
operationId: getProductTargets
parameters:
- name: searchParameters
description: "Some description"
in: query
required: true
schema:
$ref: "..ProductTargetsSearch"
responses:
200:
description: "Some description"
content:
application/json:
schema:
$ref: "..GetProductTargetsResponse"
400:
description: "Some description"
content:
application/json:
schema:
$ref: "..ErrorDto"
500:
description: "Ошибка на сервере"
content:
application/json:
schema:
$ref: "..ErrorDto"
Generator emits type for queryParams:
export type GetProductTargetsQueryParams = {
/**
* Some description
*/
searchParameters: Schemas.ProductTargetsSearch;
};
But by spec https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.0.0.md#parameter-object
parameter in: query has default value of style - form. So explode default value is true for form style. Spec says:
When this is true, parameter values of type array or object generate separate parameters for each value of the array or key-value pair of the map
Should generator emit queryParam type with spreaded params? if no, why?
- 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 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
fabien0102/openapi-codegen#356 ·
-
update dependencies & removed unused deprecatedPosiblemente ocupada @el-j la tomó hace 2 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
fabien0102/openapi-codegen#354 ·
-
deepMerge mutates its arguments, corrupting request payloads across callsPosiblemente ocupada @richard-willis-chevin la tomó hace 46 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
fabien0102/openapi-codegen#349 ·
Todos los issues de fabien0102/openapi-codegen
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
MystenLabs/MemWal#1163 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Mondriaan
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
knaw-huc/textannoviz#709 ·
Los mantenedores suelen responder en 1 día
-
billion-context-pi
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ranxianglei/billion-context#2521 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Add: YRF Music NepalAbiertostreams:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100