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

SearchKit - Default to search for individuals #23385

Merged
merged 1 commit into from May 10, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented May 6, 2022

Overview

When clicking "+ New Search" in Search Kit, default to search for Individuals instead of all contacts.

Comments

I think this is a more user-friendly start to a new search because:
a) searching for individuals is more common than all contacts
b) it demonstrates how to use the where clause

Technical Details

Probably overkill but I added a guard to prevent errors in case the "Individual" contact type is disabled.

@civibot
Copy link

civibot bot commented May 6, 2022

(Standard links)

@civibot civibot bot added the master label May 6, 2022
@seamuslee001 seamuslee001 added the merge ready PR will be merged after a few days if there are no objections label May 9, 2022
@seamuslee001
Copy link
Contributor

This also feels consistent with advanced search as well AFAIK. I'm going to add merge ready just in case anyone wants to speak up on this but this seems good to me.

@yashodha
Copy link
Contributor

@seamuslee001 @colemanw looks good, merging this. Thanks!

@yashodha yashodha merged commit ed7aab3 into civicrm:master May 10, 2022
@colemanw colemanw deleted the defaultToIndividual branch May 10, 2022 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
3 participants