-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[typescript-operations] Remove inputmaybe and scalars #10509
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master-next
Are you sure you want to change the base?
[typescript-operations] Remove inputmaybe and scalars #10509
Conversation
🦋 Changeset detectedLatest commit: bbaed46 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
packages/plugins/typescript/operations/src/ts-operation-variables-to-object.ts
Outdated
Show resolved
Hide resolved
packages/plugins/typescript/operations/src/ts-operation-variables-to-object.ts
Outdated
Show resolved
Hide resolved
|
Thank you! Looking like it's on the right track! There are just a few minor missing things:
|
|
Thank you for your help! |
|
I have added the changes that we discussed. Could you please check?
|
Description
We want to decoupe typescipt-operations from typescript plugin.
This PR removes InputMaybe and Scalars from generated variables.
Related #10496
Details
packages/plugins/typescript/operations/src/ts-operation-variables-to-object.tsgetScalaris an override to render normal typescript types instead ofScalars[...][...]. It takes into accountconfig.scalarswrapMaybeandclearOptionalfunctions are complementary: one appends a suffix; the other strips it.clearOptionalfunction visibility inpackages/plugins/typescript/typescript/src/typescript-variables-to-object.tsso it is possible to override it.Type of change
How Has This Been Tested?