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

🐛[NEXT] 表格导出当只row有一行时返回空数据 #2681

Closed
2 of 5 tasks
Volankey opened this issue Apr 24, 2024 · 2 comments · Fixed by #2682
Closed
2 of 5 tasks

🐛[NEXT] 表格导出当只row有一行时返回空数据 #2681

Volankey opened this issue Apr 24, 2024 · 2 comments · Fixed by #2682
Assignees
Labels
🐛 bug 这个是一个 bug next 2.0-next 版本的问题 released on @next

Comments

@Volankey
Copy link
Contributor

🏷 Version

Package Version
@antv/s2 2.0.0-next.17

Sheet Type

  • PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet
  • EditableSheet

🖋 Description

表格导出,当数据只有一行时,返回空数据

⌨️ Code Snapshots

while (
deadline.timeRemaining() > 0 &&
count > 0 &&
rowIndex < rowLength - 1
) {
for (let j = rowIndex; j < rowLength && count > 0; j++) {
const rowData = this.displayData[j];
const row: string[] = [];

看起来97行,rowIndex < rowLength - 1 应该是 rowIndex <= rowLength - 1

🔗 Reproduce Link

https://codesandbox.io/p/sandbox/boring-tamas-jd3hqk?file=%2Findex.ts%3A101%2C17

🤔 Steps to Reproduce

😊 Expected Behavior

应该导出正确的 csv 不是空数据

😅 Current Behavior

💻 System information

Environment Info
System
Browser
@lijinke666 lijinke666 added the next 2.0-next 版本的问题 label Apr 25, 2024
@lijinke666 lijinke666 self-assigned this Apr 25, 2024
@lijinke666 lijinke666 added the 🐛 bug 这个是一个 bug label Apr 25, 2024
Copy link
Contributor

你好 @Volankey,很抱歉给你带来了不好的体验, 我们会尽快排查问题并修复, 请关注后续发布日志.

Hello, @Volankey, We are so sorry for the bad experience. We will troubleshoot and fix the problem as soon as possible. Please pay attention to the follow-up change logs.

@lijinke666 lijinke666 linked a pull request Apr 25, 2024 that will close this issue
5 tasks
lijinke666 added a commit that referenced this issue Apr 26, 2024
* fix: 修复只有一行数据时导出数据为空 close #2681

* fix: 增加超时判断
@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v2.0.0-next.18 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 这个是一个 bug next 2.0-next 版本的问题 released on @next
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants