openapi-codegen-ts icon indicating copy to clipboard operation
openapi-codegen-ts copied to clipboard
verified publisher icon pagopa

Oas3 support request body

Open ioggstream opened this issue 6 years ago • 4 comments

This PR

begin support request body on OAS3

ioggstream avatar Nov 07 '19 18:11 ioggstream

This pull request introduces 1 alert when merging 59e32e7a80be20caaed84065b7ab435ab17880d9 into 8c6e05b171d37c7d73f2eb99ad043cc9038892a4 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Nov 07 '19 18:11 lgtm-com[bot]

This pull request introduces 1 alert when merging d9205d7ae5966a3f80566d54dbb0c6607eb3de63 into cf559868e8080dfe0e50b40440252fd56d8b05c8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Nov 10 '19 16:11 lgtm-com[bot]

hi @ioggstream, may you remind me why is this a draft and the status of open PR that can be safely merged into this repo:

https://github.com/teamdigitale/io-utils/pulls?utf8=%E2%9C%93&q=is%3Aopen+is%3Apr+-author%3Aapp%2Fdependabot-preview+

cc @francescopersico

probably we should bump version to a major release.

gunzip avatar Jan 05 '20 14:01 gunzip

Ideally there are other patches to merge first, and this one should probably be rebased.

Practically, we should merge Aymen's branch first, and then this one.

It's draft because I worked on this alone, and a review is needed.

If you're not experiencing issues on io-onboarding probably this code is worth reviewing and eventually merging.

This will enable us to implement the PATCH support.

ioggstream avatar Jan 05 '20 18:01 ioggstream