Skip to content

Feature swagger auto generate option#43

Open
thiagoarioli wants to merge 3 commits into
JKHeadley:masterfrom
thiagoarioli:FEATURE_swagger_auto_generate_option
Open

Feature swagger auto generate option#43
thiagoarioli wants to merge 3 commits into
JKHeadley:masterfrom
thiagoarioli:FEATURE_swagger_auto_generate_option

Conversation

@thiagoarioli

Copy link
Copy Markdown
Contributor

Prevent Swagger documentation generate on CRUD

@JKHeadley

JKHeadley commented Sep 3, 2017

Copy link
Copy Markdown
Owner

@thiagoarioli Looks great! However it looks like setting enableReadSwagger to false doesn't remove the getAll association endpoint, are you seeing that?

Also, what do you think about an enableSwagger option that can disable swagger for all endpoints in the model?

@JKHeadley

Copy link
Copy Markdown
Owner

Nevermind, I see you added the option in the association definition, which makes more sense.

@thiagoarioli

Copy link
Copy Markdown
Contributor Author

So have some parameter that doesn't work? The get All scope you have to pass enableReadSwagger false on the association.

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

Successfully merging this pull request may close these issues.

2 participants