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

Suggestions for improvement #161

Open
samirshaik opened this issue May 24, 2018 · 0 comments
Open

Suggestions for improvement #161

samirshaik opened this issue May 24, 2018 · 0 comments

Comments

@samirshaik
Copy link

samirshaik commented May 24, 2018

The plugin is good but could be better with following features:

  1. Ability to describe service class and methods - At the moment there is no way to add description for the generated Service class and methods. We have home grown swagger generation tool that does what we need but takes a longer route of first generating javadoc, then generating WADL and then generating swagger. By doing this we are able to pick up the java doc at the method level and add it as description in the generated definition. We don't mind adding swagger annotations like below if that could be used as standardization:
    At Class Level: @Api(value = "customers", description = "RESTful API to interact with application resources.")
    At Method Level: @ApiOperation(value = "Update Application Status", notes = "Update application status for the given application id", responseContainer = "", response = Boolean.class)

  2. All of our service methods return a wrapper object with actual response composed in that wrapper. When swagger definition is created it would be great if we could link actual object that would be composed in the wrapper. At the moment we do this by declaring javadoc as follows: @return WrapperResponse - {@link com.sample.platform.booking.entity.BookApplication com.sample.platform.booking.entity.BookApplication}

  3. Give option to select only finite set of URIs that have to be generated in the final swagger file. We have a file called include-rest-api.txt where the service URIs could be specified and the swagger generated will only have those many definitions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant