Skip to content

[client] update ScalarConverter to accept any objects #819

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

Merged
merged 2 commits into from
Aug 26, 2020

Conversation

dariuszkuc
Copy link
Collaborator

📝 Description

Custom GraphQL scalars can be serialized as any of the other built-in scalars (i.e. int, float, etc). Changed method signatures to accept Any when converting to scalar, and produce Any when generating JSON value.

🔗 Related Issues

Resolves #816

@dariuszkuc dariuszkuc added type: bug Something isn't working changes: major Changes require a major version labels Jul 30, 2020
@dariuszkuc
Copy link
Collaborator Author

Verified locally with examples/client projects - will update those once we release a version.

@smyrick smyrick added this to the 4.0.0 milestone Jul 30, 2020
Custom GraphQL scalars can be serialized as any of the other built-in scalars (i.e. int, float, etc). Changed method signatures to accept `Any` when converting to scalar, and produce `Any` when generating JSON value.
@smyrick smyrick merged commit 1804e6a into ExpediaGroup:master Aug 26, 2020
smyrick pushed a commit to smyrick/graphql-kotlin that referenced this pull request Sep 3, 2020
smyrick pushed a commit to smyrick/graphql-kotlin that referenced this pull request Sep 18, 2020
* [client] update ScalarConverter to accept any objects

Custom GraphQL scalars can be serialized as any of the other built-in scalars (i.e. int, float, etc). Changed method signatures to accept `Any` when converting to scalar, and produce `Any` when generating JSON value.

* update docs to match change to any
huehnerlady pushed a commit to huehnerlady/graphql-kotlin that referenced this pull request Oct 16, 2020
* [client] update ScalarConverter to accept any objects

Custom GraphQL scalars can be serialized as any of the other built-in scalars (i.e. int, float, etc). Changed method signatures to accept `Any` when converting to scalar, and produce `Any` when generating JSON value.

* update docs to match change to any
@dariuszkuc dariuszkuc deleted the scalar_converter branch October 23, 2020 20:52
benimario added a commit to benimario/graphql-kotlin that referenced this pull request Oct 30, 2020
dariuszkuc added a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
* [client] update ScalarConverter to accept any objects

Custom GraphQL scalars can be serialized as any of the other built-in scalars (i.e. int, float, etc). Changed method signatures to accept `Any` when converting to scalar, and produce `Any` when generating JSON value.

* update docs to match change to any
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes: major Changes require a major version type: bug Something isn't working
Development

Successfully merging this pull request may close these issues.

Add support for ScalarConverter from other types than String
2 participants