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

feat: new sortUndefined last and first options #5486

Merged
merged 1 commit into from
Apr 13, 2024
Merged

Conversation

KevinVandy
Copy link
Member

@KevinVandy KevinVandy commented Apr 13, 2024

This pr gives new "official" options to force undefined column values to either the bottom or top rows of a table when a sort is performed on those columns when the sortUndefined is set to 'last' or 'first' in the column option.

This used to be behavior out of the box with sortUndefined: 1, but was changed 10 months ago to not count undefined values. This PR will not have any breaking changes, but will allow those who preferred the old behavior more with an explicit option.

Copy link

nx-cloud bot commented Apr 13, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7e965ec. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@KevinVandy KevinVandy merged commit d2ae539 into main Apr 13, 2024
2 checks passed
@KevinVandy KevinVandy deleted the fix-sort-undefined branch April 13, 2024 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant