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.

Sign up for GitHub

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

Jump to bottom

excel 导入报错没有抛异常到前端 #672

Closed
Tracked by #703
wklken opened this issue Sep 14, 2022 · 2 comments
Closed
Tracked by #703

excel 导入报错没有抛异常到前端 #672

wklken opened this issue Sep 14, 2022 · 2 comments
Assignees
@wklken
Labels
Layer: api Api module related Type: bug Something isn't working Type: enhancement Enhancement for existing feature

Comments

@wklken
Copy link
Collaborator

wklken commented Sep 14, 2022

No description provided.

@wklken wklken self-assigned this Sep 14, 2022
@wklken wklken added Type: bug Something isn't working Type: enhancement Enhancement for existing feature Layer: api Api module related labels Sep 19, 2022
@wklken wklken added this to the Y2022M38 milestone Sep 19, 2022
@wklken
Copy link
Collaborator Author

wklken commented Sep 20, 2022

https://github.com/wklken/bk-user/blob/675d75e3c964059feb5239a5ee44c0c807042470/src/api/bkuser_core/categories/plugins/local/syncer.py#L171

目前的处理逻辑:

  1. 遍历
  2. 某条失败, 打日志, 跳过
  3. 执行完成成功

没有机制可以做一个summary, 需要加个机制

成功多少个, 失败多少个, 失败的是哪些

@wklken wklken removed this from the Y2022M38 milestone Sep 20, 2022
@wklken wklken mentioned this issue Sep 27, 2022
Closed
14 tasks
@wklken
Copy link
Collaborator Author

wklken commented Oct 9, 2022

#710

版本2.5.x中修复

@wklken wklken closed this as completed Oct 9, 2022
This was referenced Oct 9, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wklken wklken

Labels
Layer: api Api module related Type: bug Something isn't working Type: enhancement Enhancement for existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wklken