Skip to content

[Dart][Jaguar] various improvements #1596

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 1 commit into from
Dec 6, 2018

Conversation

jaumard
Copy link
Contributor

@jaumard jaumard commented Dec 2, 2018

Update dart deps
Add default timeout on http requests
Add nullableField option to send or not null field on the payload

add properties to send or not null values
update deps
@jaumard jaumard force-pushed the feature/update_jaguar_gen branch from 779fbae to 5107909 Compare December 2, 2018 15:23
@wing328
Copy link
Member

wing328 commented Dec 3, 2018

Thanks for the PR. Please update the PR with a description to explain the change and a better title/subject to describe the PR.

@jaumard jaumard changed the title Feature/update jaguar gen Feature/jaguar generator improvements Dec 3, 2018
@jaumard
Copy link
Contributor Author

jaumard commented Dec 3, 2018

@wing328 done :)

@@ -2,12 +2,12 @@ part '{{classFilename}}.jser.dart';

class {{classname}} {
{{#vars}}{{#description}} /* {{{description}}} */{{/description}}
@Alias('{{{baseName}}}')
@Alias('{{baseName}}')
Copy link
Member

Choose a reason for hiding this comment

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

Minor suggestion: using {{{baseName}}} to avoid escaping HTML special characters.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure :) my bad ^^

@wing328 wing328 merged commit d40cbf9 into OpenAPITools:master Dec 6, 2018
@jaumard jaumard deleted the feature/update_jaguar_gen branch December 6, 2018 10:44
@wing328 wing328 changed the title Feature/jaguar generator improvements [Dart][Jaguar] various improvements Dec 6, 2018
@wing328
Copy link
Member

wing328 commented Jan 2, 2019

@jaumard thanks for the contribution, which has been included in the 4.0.0-beta release: https://twitter.com/oas_generator/status/1079727020374806529.

Happy New Year and looking forward to more collaboration and contributions in 2019!

A-Joshi pushed a commit to ihsmarkitoss/openapi-generator that referenced this pull request Feb 27, 2019
add properties to send or not null values
update deps
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