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

RestfulApi增加了批量更新接口 #111

Closed
wants to merge 1 commit into from

Conversation

wlclass
Copy link
Contributor

@wlclass wlclass commented Dec 14, 2022

为RestfulApi增加了一个接口POST: /api/Config/MergeUpdate.

用来批量修改一组配置,在请求参数中传一个json字符串,后端解析kv后,与现有配置进行比较,如果配置以前存在并且此次有修改,就更新,如果配置以前不存在,则本次就添加.
同时处理好了权限,增加了相关的权限定义,如: Functions.Config_MergeUpdate

可能修复了这个#100

@kklldog
Copy link
Collaborator

kklldog commented Dec 15, 2022

你好,很好,但是这个接口很不 restful 。混合了post跟put ,url 命名有动词。这是一个明显的 rpc 风格的接口。笑哭

@kklldog kklldog closed this Dec 15, 2022
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.

None yet

2 participants