[feat][WEB] add delete functionality for configuration parameters and user config#1037
Closed
v-kkhuang wants to merge 3 commits into
Closed
[feat][WEB] add delete functionality for configuration parameters and user config#1037v-kkhuang wants to merge 3 commits into
v-kkhuang wants to merge 3 commits into
Conversation
- 新增删除按钮UI，仅管理员可见 - 实现删除确认对话框 - 调用现有接口 DELETE /configuration/keyvalue 删除配置值 - 添加前端权限控制 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
在用户配置管理页面添加删除功能，支持管理员删除用户配置行。
主要修改：
- 在用户配置管理页面操作列添加删除按钮
- 实现deleteConfig()方法，调用DELETE /configuration/baseKeyValue接口
- 添加删除确认对话框，显示配置详情
- 添加中英文国际化文本
- 修改数据映射，确保configKeyId字段可用
技术细节：
- 前端权限控制：仅管理员可查看和操作删除按钮
- 删除对象：整个用户配置行（用户+应用+引擎类型+版本+配置项）
- 删除接口：DELETE /configuration/baseKeyValue?id={configKeyId}
- 用户确认：弹窗显示删除配置的完整信息
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
v-kkhuang
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is the purpose of the change
Background/Problem:
Administrators encounter redundant configuration values in both the global parameter settings and user-specific configurations. Currently, there is no UI-based way to delete these configurations, requiring manual database operations which is inefficient and risky.
Purpose of Change:
To address this problem, this PR adds delete functionality to two areas: (1) the variable component in settings page for deleting global configuration parameters, and (2) the user configuration management page for deleting user-specific configurations. Both features include confirmation dialogs and front-end permission control.
Value/Impact:
After the change, administrators can safely delete redundant configurations through the UI with proper confirmation dialogs, improving operational efficiency and reducing the risk of manual database errors.
Related issues/PRs
Related issues: close #1038
Related pr:none
Brief change log
Checklist