search

devexperts / swagger-codegen-ts

Typesafe Swagger API generator for TypeScript
Mozilla Public License 2.0
80 stars 16 forks source link

Support headers #133

Closed Lonelind closed 3 years ago

Lonelind commented 3 years ago

Hi!

This PR adds support for headers, partly resolving #57.

Test cases with arrays also added.

raveclassic commented 3 years ago

Hi @Lonelind, thank you for the PR!

Could you please clarify what you mean by "partially resolving #57"?

raveclassic commented 3 years ago

@Lonelind Please open a satellite issue tracking swagger2.0 spec if you are not going to support it in this PR (I wouldn't suggest to do it).

Lonelind commented 3 years ago

I didn't add formData there but I guess it'd be about the same. Currently, this only helps with headers because it's really needed on my project due to the API design.

raveclassic commented 3 years ago

@Lonelind Please open issues to track missing things :)

Lonelind commented 3 years ago

Swagger 2.0 is supported

raveclassic commented 3 years ago

@Lonelind Sorry, didn't notice that! Thanks, LGTM. But please open an issue for the missing parts (FormData etc.)

Lonelind commented 3 years ago

I'd rather add a notice to #57 that headers are done and resolve #39 because this PR just adds one thing of many from the list.