Skip to content

Fix Kotlin query collection format - #4248

Merged
wing328 merged 2 commits into
OpenAPITools:masterfrom
andrewemery:kotlin-query-collection-format
Oct 29, 2019
Merged

Fix Kotlin query collection format#4248
wing328 merged 2 commits into
OpenAPITools:masterfrom
andrewemery:kotlin-query-collection-format

Conversation

@andrewemery

@andrewemeryandrewemery commented Oct 24, 2019

Copy link
Copy Markdown
Contributor

Fixes incorrect query header collection constants

The OpenAPI generator converts OpenAPI v2 documents into an OpenAPI v3
representation. The query parameter collection formats are squeezed
into different values from the raw OpenAPI 2 collection format values.

For example: csv -> space

See: DefaultCodegen#getCollectionFormat

To fix#4285

The OpenApi codegen converts OpenApi v2 documents into an OpenApi v3
representation. The query parameter collection formats are squeezed
into different values from the raw OpenApi 2 collection format values.
For example: csv -> space
See: DefaultCodegen#getCollectionFormat
@andrewemery
andrewemeryforce-pushed the kotlin-query-collection-format branch from 06bc9af to ee1ccdaCompareOctober 28, 2019 09:44
@wing328

Copy link
Copy Markdown
Member

cc @jimschubert (2017/09), @dr4ke616 (2018/08) @karismann (2019/03) @Zomzog (2019/04)

@wing328wing328 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix, which looks good to me.

@wing328
wing328 merged commit a2b37d6 into OpenAPITools:masterOct 29, 2019
@wing328wing328 changed the title Kotlin query collection formatFix Kotlin query collection formatOct 29, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] [Kotlin Client] Incorrect pipes and csv query collection delimeter

2 participants

@andrewemery@wing328