Skip to content

Expose Initializable Params in Swagger Docs#2170

Closed
ragboyjr wants to merge 1 commit intoapi-platform:masterfrom
ragboyjr:expose-intializable-params-in-swagger-docs
Closed

Expose Initializable Params in Swagger Docs#2170
ragboyjr wants to merge 1 commit intoapi-platform:masterfrom
ragboyjr:expose-intializable-params-in-swagger-docs

Conversation

@ragboyjr
Copy link
Contributor

@ragboyjr ragboyjr commented Aug 21, 2018

Any resource parameters that were initializable (but not writable) would not show up as writable in the API documentation.

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Please update this template with something that matches your PR

Signed-off-by: RJ Garcia <rj@bighead.net>
@dunglas
Copy link
Member

dunglas commented Sep 11, 2018

Duplicate of #2178. Sorry about that, we didn't noticed that both PRs were fixing the same issue before. Thank you very much for contributing!

@dunglas dunglas closed this Sep 11, 2018
@ragboyjr ragboyjr deleted the expose-intializable-params-in-swagger-docs branch September 19, 2018 00:57
@ragboyjr
Copy link
Contributor Author

Thanks @dunglas !

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