查询导出改为后台进行,bootstrap-table数据超过几百页面就会卡住#3016
Open
a516597993 wants to merge 5 commits intohhyo:masterfrom
Open
Conversation
记录查询类型
增加后台导出
增加后台导出
增加后台导出功能
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3016 +/- ##
==========================================
- Coverage 78.51% 77.84% -0.67%
==========================================
Files 125 126 +1
Lines 17822 17985 +163
==========================================
+ Hits 13993 14001 +8
- Misses 3829 3984 +155 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
LeoQuote
requested changes
Jul 23, 2025
Collaborator
LeoQuote
left a comment
There was a problem hiding this comment.
- 代码和查询部分的代码大部分重复, 如果查询逻辑和之前的在线显示的一致, 请考虑在 query 接口处新增参数, 并在前端新增适配进行实现
- query type 此处设置的没有意义, 从数据安全审查角度来看, 不管用户选择了下载还是不下载, 实际上都有数据的泄露, 不应该做区分
- 下载方式并没有根本上的改变, 标题上写了改为后台进行, 实际上依然是在一个阻塞的 http 请求中获取生成的 excel 或其他格式的数据文件, 建议修改描述或者更改下载流程, 改为分为两步完成: 用户请求查询, 获取查询 ID, 后台进行查询, 准备文件, 用户过一段时间使用 ID 进行数据文件的下载.
Collaborator
|
可参考已有的 pr #2685 , 或基于此 pr 进行修改开发 |
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
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.
查询导出改为后台进行,bootstrap-table数据超过几百页面就会卡住,记录导出行为