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

API Refactor SiteTree filtering to work like GridField filters #3052

Draft
wants to merge 1 commit into
base: 6.0
Choose a base branch
from

Conversation

GuySartorelli
Copy link
Member

CMSMain uses an entirely separate code path for building the filter form and for the actual filter functionality compared with the gridfield filter used literally everywhere else in the CMS.

This aims to unify the two as much as possible, so that filtering pages in CMSMain is the same as filtering everywhere else.
It also generalises it which will help to unlock #2951

Issue

@GuySartorelli GuySartorelli marked this pull request as draft January 30, 2025 03:15
@GuySartorelli GuySartorelli force-pushed the pulls/6.0/refactor-sitetree-filter branch 2 times, most recently from 7d991c5 to be997fd Compare February 5, 2025 04:10
@GuySartorelli GuySartorelli force-pushed the pulls/6.0/refactor-sitetree-filter branch 3 times, most recently from 44daef5 to e8297af Compare February 11, 2025 02:11
@GuySartorelli GuySartorelli force-pushed the pulls/6.0/refactor-sitetree-filter branch 6 times, most recently from ce54bb9 to 85c32a0 Compare February 25, 2025 03:47
@GuySartorelli GuySartorelli force-pushed the pulls/6.0/refactor-sitetree-filter branch from 85c32a0 to 7243da7 Compare February 25, 2025 03:51
CMSMain uses an entirely separate code path for building the filter form
and for the actual filter functionality compared with the gridfield
filter used literally everywhere else in the CMS.
This aims to unify the two as much as possible, so that filtering pages
in CMSMain is the same as filtering everywhere else.
@GuySartorelli GuySartorelli force-pushed the pulls/6.0/refactor-sitetree-filter branch from 7243da7 to cf1ff64 Compare February 25, 2025 03:53
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.

1 participant