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

Add a useFilterSortBy hook #4064

Merged
merged 1 commit into from
Jun 13, 2024
Merged

Add a useFilterSortBy hook #4064

merged 1 commit into from
Jun 13, 2024

Conversation

bjoernricks
Copy link
Contributor

What

Add a useFilterSortBy hook

Why

The hook determines the sort field and direction of a filter and allows to change both via a returned function. The hook can be used to implement the filter changes when clicking on the different header columns of a entities list.

References

Get rid of entities pages HOCs

Checklist

  • Tests

The hook determines the sort field and direction of a filter and allows
to change both via a returned function. The hook can be used to
implement the filter changes when clicking on the different header
columns of a entities list.
@bjoernricks bjoernricks requested a review from a team as a code owner June 13, 2024 09:37
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA e4efaa2.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link

Conventional Commits Report

Type Number
Added 1

🚀 Conventional commits found.

@bjoernricks bjoernricks enabled auto-merge (rebase) June 13, 2024 09:38
@bjoernricks bjoernricks merged commit 3af32a4 into main Jun 13, 2024
11 checks passed
@bjoernricks bjoernricks deleted the use-filter-sort-by branch June 13, 2024 09:45
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.

2 participants