Skip to content
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

[C#] Add support for Readonly property #2199

Merged
merged 4 commits into from
Feb 21, 2016

Conversation

wing328
Copy link
Contributor

@wing328 wing328 commented Feb 21, 2016

  • Added support for Readonly property in C#
  • Updated constructor to take optional parameters
  • Updated constructor to throw exception if the required parameter is not provided

@wing328 wing328 added this to the v2.1.6 milestone Feb 21, 2016
wing328 added a commit that referenced this pull request Feb 21, 2016
[C#] Add support for Readonly property
@wing328 wing328 merged commit ad6380c into swagger-api:master Feb 21, 2016
@wing328
Copy link
Contributor Author

wing328 commented Feb 22, 2016

@jimschubert I'll make similar change to modules/swagger-codegen/src/main/resources/aspnet5/model.mustache this week or early next week. Please let me know if you see any potential issue with the change for aspnet5

@wing328
Copy link
Contributor Author

wing328 commented Feb 26, 2016

Upgrade note from 2.1.5 to 2.1.6

Starting from 2.1.6, readOnly members (which should not be set with any value) in C# model will not have any public setter.

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