-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
新增批量操作工单 #1531
新增批量操作工单 #1531
Conversation
1.8.4 提交SQL已经支持实例配置的时候配上db_name。强烈建议支持“批量提交SQL” |
这个功能我这边已经实现了 ,只是这个需求极少数朋友才会有。所以这块功能暂未提pr |
这个功能还是有必要的,易用性会提升一大截,尽快提pr吧,谢谢。 |
Codecov Report
@@ Coverage Diff @@
## master #1531 +/- ##
==========================================
- Coverage 76.84% 75.97% -0.88%
==========================================
Files 91 91
Lines 14309 14484 +175
==========================================
+ Hits 10996 11004 +8
- Misses 3313 3480 +167
Continue to review full report at Codecov.
|
现在工单提交已经有接口了, 这个能否做成纯前端的功能? 因为后端这边做测试有点太麻烦了 |
这个代码我这边线上已经跑了快小一年了,这个后端处理会比较合适一点吧 ,我这边的需求有时候要处理1000多个工单,这样要反复调用1000多次接口 。不太合理感觉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
代码复用不够, 这些检查的代码在单个工单的审核和执行中有类似的, 希望能复用一部分, 可以考虑直接把代码挪到 model 层, 作为 workflow 的一个 方法来做
这个对多租户和一些分库分表很有用啊,我们现在提相同语句的工单在不同实例要提很多次,也要审批很多次。能发出来么 |
@cslingjun |
提交SQL按钮旁边是有个批量提交SQL工单按钮的 。你看我截图 。但是这块代码好像我没提上来 |
大佬好,可以指导一下怎么加这一块的代码吗,想要个批量提交SQL工单的功能,感谢感谢 |
是的大佬,感谢 |
具体功能看图 #1469