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 - Provide context information to subscribers #26716

Merged
merged 1 commit into from
Jul 3, 2023

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jul 3, 2023

Overview

This is a small piece broken off from #26676 which has stalled on discussion about display_name formatting, but this piece is useful elsewhere so best merge it sooner rather than later.

@civibot
Copy link

civibot bot commented Jul 3, 2023

(Standard links)

@civibot civibot bot added the master label Jul 3, 2023
@mlutfy
Copy link
Member

mlutfy commented Jul 3, 2023

I don't feel qualified enough to merge here, but as someone who recently started fiddling with the autocomplete hooks, this seems very useful.

Is civi.search.defaultDisplay documented, and if not, what would be the right place to document it?

@colemanw
Copy link
Member Author

colemanw commented Jul 3, 2023

@mlutfy yes it is documented at https://docs.civicrm.org/dev/en/latest/framework/autocomplete/ but not with this new context param. Docs could be updated to reflect that once this is merged.

@eileenmcnaughton eileenmcnaughton merged commit 7ab788a into civicrm:master Jul 3, 2023
@eileenmcnaughton eileenmcnaughton deleted the searchContext branch July 3, 2023 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants