Skip to content
This repository was archived by the owner on Feb 9, 2026. It is now read-only.

Extend provider configuration with new options and add ability to filter autocomplete results before rendering#25

Merged
nicholasio merged 3 commits into10up:developfrom
lukaspawlik:feature/search-ssr
Oct 19, 2021
Merged

Extend provider configuration with new options and add ability to filter autocomplete results before rendering#25
nicholasio merged 3 commits into10up:developfrom
lukaspawlik:feature/search-ssr

Conversation

@lukaspawlik
Copy link
Copy Markdown
Contributor

Description of the Change

This PR contains changes that:

  • allow control Autocomplete / Search a bit more by exposing options to configure EP endpoint
  • modify autocomplete result before rendering in the list
  • fixes some issues with initial state when Autocomplete is also used as default input for classic website search

Alternate Designs

Benefits

Projects will be able to better adjust ElasticPress React integration configuration and UI behaviour.

Possible Drawbacks

Verification Process

Introduced changes were tested on a client project and their behaviour (results links change, initial search term state, modified EP endpoint) was validated correctly.

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Applicable Issues

Changelog Entry

Copy link
Copy Markdown
Member

@nicholasio nicholasio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@nicholasio nicholasio merged commit 631722d into 10up:develop Oct 19, 2021
@lukaspawlik lukaspawlik deleted the feature/search-ssr branch November 19, 2021 15:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants