Skip to content

[Java] using java.util.UUID for UUIDs#4739

Merged
wing328 merged 1 commit intoswagger-api:masterfrom
MatanRubin:uuid-type
Feb 8, 2017
Merged

[Java] using java.util.UUID for UUIDs#4739
wing328 merged 1 commit intoswagger-api:masterfrom
MatanRubin:uuid-type

Conversation

@MatanRubin
Copy link
Copy Markdown
Contributor

Before this change if a model had a property of type=string and
format=uuid, it would be generated as a Java String.
Now, it generates the property as java.util.UUID.

Before this change if a model had a property of type=string and
format=uuid, it would be generated as a Java String.
Now, it generates the property as java.util.UUID.
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Feb 8, 2017

@MatanRubin thanks for the PR. Since this's impacting Java server generator as well, I'll need a bit more time to review.

@wing328 wing328 merged commit 1339222 into swagger-api:master Feb 8, 2017
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Feb 8, 2017

@MatanRubin I've regenerated the samples related to Java client/server and the result looks good (CIs passed).

Thanks for your contribution.

@MatanRubin MatanRubin deleted the uuid-type branch February 8, 2017 19:38
@MatanRubin
Copy link
Copy Markdown
Contributor Author

Thanks @wing328!

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Feb 20, 2017

Linked to #1806

@wing328 wing328 changed the title [Java][Issue #1806] generate using java.util.UUID for UUIDs [Java] generate using java.util.UUID for UUIDs Feb 20, 2017
@wing328 wing328 changed the title [Java] generate using java.util.UUID for UUIDs [Java] using java.util.UUID for UUIDs Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants