-
Notifications
You must be signed in to change notification settings - Fork 119
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
Issues found after trying to generate big swagger file #58
Comments
Thanks for your feedback. Could you please share swagger.json? We tested it on more than 100 swagger files and everything was OK. Also can send it to ysampir@gmail.com if you don't want to share it with others. about:
And again, big thanks for your review and testing! @Vovanella95 could you please handle points above? |
Hi @szotp, About point 2. To have Future at response, you need to have "boolean" return type in response. I guess you have "bool" now. But according spec there is no "bool" base type https://swagger.io/specification/. There is only "boolean". Please correct me if I wrong. |
Hi, thanks for feedback, I will prepare minimal swagger spec to illustrate the problems. |
@szotp any progress? |
Sorry I had some work to do during the weekend, I'll do it this week. |
@szotp Hey, Points 3, 4 and 6 are already fixed! |
Here is the json, some issues still remain. Good luck!
|
Hi @szotp , We fixed all issues in 1.2.0 version. Refactoring issue is moved to separate one. In future we're going to implement native requests (not using chopper), but not right now :) |
Hi, I like the idea of dart only code generator, but unfortunately, I found a lot of issues while trying to generate code for my API:
final null foo;
variables. At least it should usefinal null foo;
.Future<Response<Bool>>
. It seems that onlynum
primitive is handled properly.The text was updated successfully, but these errors were encountered: