Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Scalar type for UUID#18

Merged
bbakerman merged 1 commit into
graphql-java:masterfrom
ravikancherla:master
Feb 25, 2022
Merged

Scalar type for UUID#18
bbakerman merged 1 commit into
graphql-java:masterfrom
ravikancherla:master

Conversation

@ravikancherla

Copy link
Copy Markdown

A new scalar type of id - UUID is added

  • Accepts string value or UUID value that will be converted to java.util.UUID at runtime
  • Added groovy test cases
  • Updated documentation

Relates to #4

@ravikancherla

Copy link
Copy Markdown
Author

@bbakerman - Any idea when this PR can be looked into ?

@ketiko

ketiko commented Jun 3, 2020

Copy link
Copy Markdown

This would be great to have.

@plaurina

plaurina commented Dec 2, 2020

Copy link
Copy Markdown

Bump.

Wanting to make sure I'm helping to show there is public interest in getting this merged in.
Thank you @ravikancherla for coding it up - I am currently using it.

This PR has been open for a year, if there's any of requirements for getting this approved that I can help with I'd be open to working to get it to meet them if @ravikancherla has disappeared/lost interest and the admins want help to move this along.

@ravikancherla

Copy link
Copy Markdown
Author

@plaurina - as far as I know I have passed all the checks and have been waiting for admins to merge this PR for over an year 😊

Of course with conflicts because of new changes happened after that. Feel free to resolve the conflicts. I think that’s all is needed from our side and admins have to merge this PR after that

@guissalustiano guissalustiano left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGT

@bbakerman bbakerman mentioned this pull request Feb 24, 2022
@bbakerman bbakerman merged commit ea81b1f into graphql-java:master Feb 25, 2022
@bbakerman

Copy link
Copy Markdown
Member

technically via #55 - but it started from this code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants