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

perf: 更新文件源接口只传id不传code时报错 #2414 #2537

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

liuliaozhong
Copy link
Collaborator

No description provided.

@@ -202,6 +202,7 @@
1260016=At least one of id/code required
1260017=FileSource[id={0}] does not belong to specified business
1260018=FileWorker response error, Detail:{0}
1260019=File source code can not empty
Copy link
Collaborator

Choose a reason for hiding this comment

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

can not be empty

@@ -325,6 +325,8 @@ public class ErrorCode {
public static final int FILE_SOURCE_ID_NOT_IN_BIZ = 1260017;
// 接入点响应异常,详情:{0}
public static final int FAIL_TO_REQUEST_FILE_WORKER_WITH_REASON = 1260018;
// 文件源code不可为空
public static final int FILE_SOURCE_CODE_NOT_EMPTY = 1260019;
Copy link
Collaborator

Choose a reason for hiding this comment

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

名称含义不清晰,改为:FILE_SOURCE_CODE_CAN_NOT_BE_EMPTY

Copy link
Collaborator

@jsonwan jsonwan left a comment

Choose a reason for hiding this comment

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

  1. 2个小问题需要处理;
  2. 代码没问题后,请在新的联合开发环境上提交一个测试用的验证作业。

@jsonwan jsonwan merged commit 0531a44 into TencentBlueKing:3.7.x Dec 12, 2023
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.

2 participants