-
Notifications
You must be signed in to change notification settings - Fork 339
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
doc: add apisix_pluginconfig document #1025
Conversation
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.
thanks
You should modify this file also https://github.com/apache/apisix-ingress-controller/blob/master/docs/en/latest/config.json
ok |
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.
LGTM
| plugins | array | A series of custom plugins that will be executed once this route rule is matched | | ||
| plugins[].name | string | The plugin name, see [docs](http://apisix.apache.org/docs/apisix/getting-started) for learning the available plugins. | | ||
| plugins[].enable | boolean | Whether the plugin is in use | | ||
| plugins[].config | object | The plugin configuration, fields should be same as in APISIX. | |
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.
| plugins[].config | object | The plugin configuration, fields should be same as in APISIX. | | |
| plugins[].config | object | The plugin configuration, and fields should be the same as in APISIX. | |
|---------------|----------|----------------------------------------------------| | ||
| plugins | array | A series of custom plugins that will be executed once this route rule is matched | | ||
| plugins[].name | string | The plugin name, see [docs](http://apisix.apache.org/docs/apisix/getting-started) for learning the available plugins. | | ||
| plugins[].enable | boolean | Whether the plugin is in use | |
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.
| plugins[].enable | boolean | Whether the plugin is in use | | |
| plugins[].enable | boolean | Whether the plugin should be enabled | |
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.
@neverCase PTAL
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.
I will merge first, for those small suggestions @neverCase could help to create a new PR :)
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.
ok, i will open another one
Type of change:
What this PR does / why we need it:
issue: #638
Pre-submission checklist: