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

Pl 10 add config option to allow for empty search #64

Merged
merged 3 commits into from
Dec 16, 2018

Conversation

mcarmichael21
Copy link
Contributor

@mcarmichael21 mcarmichael21 commented Dec 4, 2018

PL-10

Ticket Description

Add "Show results for empty search" (empty-search-results or something) option to federated-search-react and create the corresponding config option search_api_federated_solr D8 and D7 branches. This should be a boolean that defaults to false.
Reconfigure react components to behave based on that config.

PR Description

This PR adds the show_empty_search_results config to the drupal settings form for federated search

To Test (from federated-search-react repo)

  1. Dependent on Pl 10 add config option to allow for empty search federated-search-react#25

@froboy
Copy link
Contributor

froboy commented Dec 16, 2018

LGTM!

@froboy froboy merged commit 38d3deb into master Dec 16, 2018
@froboy froboy deleted the PL-10-Add-config-option-to-allow-for-empty-search branch December 18, 2018 18:29
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