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

优化 AI 内容安全插件参数示例 #342

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Conversation

mrhaoji
Copy link
Contributor

@mrhaoji mrhaoji commented Sep 24, 2024

默认展示是否启用的参数

Ⅰ. Describe what this PR did

AI 内容安全插件展示的默认参数里面没有带是否启用的参数
造成不了解完整参数的用户在启用该插件后会有插件没有生效的错误感知。

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

easy to understand

Ⅳ. Describe how to verify it

just a example which show to the user, no need to verify anything

Ⅴ. Special notes for reviews

默认展示是否启用的参数
@CLAassistant
Copy link

CLAassistant commented Sep 24, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@CH3CHO CH3CHO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CH3CHO
Copy link
Collaborator

CH3CHO commented Sep 24, 2024

请签署 CLA。谢谢~

@CH3CHO CH3CHO merged commit b563984 into higress-group:main Sep 25, 2024
4 checks passed
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.

3 participants