Issues with generic definitions about swagger-to-flowtype

[TheTFo]

I checked at what https://github.com/swagger-api/swagger-codegen generates for typescript, they basically just remove the generic nature:

export interface IPageCustomer
export interface IPageUser

I can make this change, are you accepting PR's?

[yayoc]

This generator seems to just replace [ and ] with < and > and typescript also doesn't allow to declare duplicate generics. At this time, I think replacing [ and ] with < and > would be enough for this library.

[TheTFo]

The issue I ran into this is that flow throws errors on the generated flowtype.js that we have multiple exports of the interface. Maybe flow will support this in the future?

…

On Mon, Jun 26, 2017, 10:23 AM yayoc ***@***.***> wrote: This generator <https://github.com/areijngoudt/swagger-ts-generator/blob/876e52e34ad2708c9a26ac1106c04ec804b06f13/lib/modelGenerator.js#L318-L320> seems to just replace [ and ] with < and > and typescript also doesn't allow to declare duplicate generics. At this time, I think replacing [ and ] with < and > would be enough for this library. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAgWoEI3FRnZ1n7U_Xf-fU3SQJ3GRoIrks5sH77EgaJpZM4OEwzY> .

[yayoc]

Yes, please create PR.
@TheTFo

[TheTFo]

Pull Request #4 waiting to be reviewed.