-
Notifications
You must be signed in to change notification settings - Fork 41
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
Setting up "Models" section #12
Comments
Hi, Generics classes are unsupported. class NotGeneric: GenericClass<Generic>() This should allow the reflection to find the appropriate types. |
As for the nulls, the library currently relies on ktor to serialize the model into json. The |
Although thinking about it, the KType information seems to have the generic information... Maybe there is a way to handle it. |
Also, there is a hook in the configuration that allows you to properly rename the classes to be digestible by the standard. |
@SerVB I changed the model acquisition system in the new release. Please verify if your issue persists or not. |
https://github.com/papsign/Ktor-OpenAPI-Generator/releases/tag/0.1-beta.1 Fixed. Marking as closed. |
@Wicpar, thank you! I see many changes, for example, However, there are things that I still don't understand:
If I do
So is there a way to annotate it? |
these are separate issues... |
Hi! Is there an opportunity to change the "Models" section?
Currently, I have the following one (on this commit):
I'm not an experienced Swagger user but I think the issues are:
T
model should not be presented.minimum: null/maximum: null
forstring
andboolean
seem redundant.minimum: -2147483648
forid: int32
isn't appropriate, I want it to start with0
.kotlin.collections.List
is strange, it's a simple JSON array...io.github.servb.eShop.util.OptionalResult<io.github.servb.eShop.route.product.v1.ProductUsable>
has thedata: T
field, butdata: io.github.servb.eShop.route.product.v1.ProductUsable
is more logical here.I've briefly checked the annotations but haven't found suitable ones.
The text was updated successfully, but these errors were encountered: