-
Notifications
You must be signed in to change notification settings - Fork 14.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
fix: Keep chosen columns sort option when changing a column #15918
Conversation
Codecov Report
@@ Coverage Diff @@
## master #15918 +/- ##
==========================================
- Coverage 77.01% 76.99% -0.02%
==========================================
Files 986 986
Lines 51978 51983 +5
Branches 7091 7091
==========================================
- Hits 40030 40025 -5
- Misses 11722 11732 +10
Partials 226 226
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
nice!
/testenv up |
@junlincc Ephemeral environment spinning up at http://52.10.94.110:8080. Credentials are |
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! Thanks for the quick fix! 🙇♀️
Ephemeral environment shutdown and build artifacts deleted. |
…5918) * Keep sorting * Fix adding new column
…5918) * Keep sorting * Fix adding new column
…5918) * Keep sorting * Fix adding new column
…5918) * Keep sorting * Fix adding new column
SUMMARY
This PR keeps the existing columns sort option while the user makes changes to the columns. Previously the sort order was not kept.
Fixes: #15842
BEFORE
126688420-181454e1-ab8d-4682-bb6e-763005888c1d.mov
AFTER
DEV.Age.distri.mp4
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION