-
-
Notifications
You must be signed in to change notification settings - Fork 665
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
fix: copy sheet should copy filter #3592
Conversation
View Deployment
|
Playwright test resultsDetails 8 tests across 3 suites |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #3592 +/- ##
==========================================
+ Coverage 31.84% 31.86% +0.02%
==========================================
Files 2194 2194
Lines 115259 115343 +84
Branches 25383 25407 +24
==========================================
+ Hits 36706 36759 +53
- Misses 78553 78584 +31
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@ybzky 设置筛选后删除再undo,筛选条件未保留 |
@ybzky After setting the filter, delete it and then undo, the filter conditions are not retained. |
ef130f8
to
b69960a
Compare
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.
LGTM
https://github.com/dream-num/univer-pro/issues/2232 |
https://github.com/dream-num/univer-pro/issues/2232 |
close https://github.com/dream-num/univer-pro/issues/940 https://github.com/dream-num/univer-pro/issues/2624 https://github.com/dream-num/univer-pro/issues/2210 https://github.com/dream-num/univer-pro/issues/2232
Pull Request Checklist