redux-toolkit icon indicating copy to clipboard operation
redux-toolkit copied to clipboard

support default return schemas in openapi codegen

Open barp opened this issue 1 year ago • 12 comments

Accidently closed PR, reopening it here.

barp avatar Feb 03 '24 08:02 barp

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders
Open Preview

codesandbox[bot] avatar Feb 03 '24 08:02 codesandbox[bot]

Deploy Preview for redux-starter-kit-docs ready!

Name Link
Latest commit 7d5fa420822377fab00cd62e307e58de2c5a6c74
Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/65be3ede92e7840007fa9e11
Deploy Preview https://deploy-preview-4152--redux-starter-kit-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Feb 03 '24 08:02 netlify[bot]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 7d5fa420822377fab00cd62e307e58de2c5a6c74:

Sandbox Source
Vanilla Typescript Configuration
rsk-github-issues-example Configuration
@examples-query-react/basic Configuration
@examples-query-react/advanced Configuration
@examples-action-listener/counter Configuration
rtk-esm-cra Configuration

codesandbox-ci[bot] avatar Feb 03 '24 08:02 codesandbox-ci[bot]

Can someone look at this?

barp avatar Feb 10 '24 09:02 barp

Bump

barp avatar Feb 14 '24 03:02 barp

@phryneas Hi, you looked at this before can you continue the review?

barp avatar Feb 21 '24 11:02 barp

@EskiMojo14 you also looked at this, can you continue the review?

barp avatar Feb 25 '24 02:02 barp

I also need this feature, it would be great if someone release it

iliapnmrv avatar Feb 25 '24 19:02 iliapnmrv

bump for this feature to be implemented

sqkrv avatar Feb 26 '24 21:02 sqkrv

Bumping again to try and get some attention...

barp avatar Mar 02 '24 23:03 barp

It will take as long as it will take.

We all work on this voluntarily, and none of us uses the codegen, or anything using OpenAPI at all. Every few months, someone will sit down, go through the OpenAPI PR, tries to understand what features they are for, if they make sense and if they might be breaking the experience for other users, and gets to merging (if applicable).

Honestly, it's not really a "fun" activity, it's one we do because we don't want to leave people hanging. Regular pings don't make it a more attractive task, though.

In the meantime, CodeSandbox has created an npm package you can use without a local patch: grafik grafik

phryneas avatar Mar 03 '24 15:03 phryneas

Thanks for giving a details explanation on how to use the temporary packages, appreciate it.

barp avatar Mar 05 '24 11:03 barp