-
Notifications
You must be signed in to change notification settings - Fork 100
OpenAPI CRD user guide doc #664
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
Conversation
be30cb2 to
a85fe4e
Compare
|
/retest |
doc/openapi-user-guide.md
Outdated
| ## OpenAPI document sources | ||
|
|
||
| The OpenAPI document <OAS> can be read from different sources: | ||
| * Openshit secret |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo(:eyes:) and it should be Kubernetes Secret
doc/openapi-user-guide.md
Outdated
| fields are provided in the [OpenAPI custom resource](openapi-reference.md), | ||
| the product's deployment mode will be `APIcast self-managed`. | ||
|
|
||
| Example of a [OpenAPI custom resource](openapi-reference.md) with custom public base URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing : at the end
miguelsorianod
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 , just a couple minor comments
|
requested changes applied. |
|
Code Climate has analyzed commit f334e44 and detected 0 issues on this pull request. View more on Code Climate. |
Explained how 3scale is configured from the OpenAPI document. Which parts of the OpenAPI document are used, and how, to configure 3scale.
Also added documentation about the minimum OpenAPI Document requried to configure a working 3scale product.
https://issues.redhat.com/browse/THREESCALE-4712