Skip to content

Conversation

@bluefuton
Copy link
Contributor

Changes proposed in this Pull Request:

In #14247 we're merging our instant-search-master branch back into Jetpack master. As part of the review process, we've made a number of fixes. This PR adds those fixes to the instant-search-master branch.

Is this a new feature or does it add/remove features to an existing part of Jetpack?

No - this pulls fixes from another PR into the instant-search-master.

Testing instructions:

  1. If you haven't already, set up Instant Search following the instructions in the readme.
  2. Enter a search query into your site's search input. Ensure that the search results render onto your site's main content area.

Proposed changelog entry for your changes:

Not required.

@bluefuton bluefuton requested a review from a team December 19, 2019 02:31
@bluefuton bluefuton self-assigned this Dec 19, 2019
@bluefuton bluefuton changed the base branch from master to instant-search-master December 19, 2019 02:31
@bluefuton bluefuton removed the request for review from a team December 19, 2019 02:31
@jetpackbot
Copy link
Collaborator

Warnings
⚠️

pre-commit hook was skipped for one or more commits

This is an automated check which relies on PULL_REQUEST_TEMPLATE. We encourage you to follow that template as it helps Jetpack maintainers do their job. If you think 'Testing instructions' or 'Proposed changelog entry' are not needed for your PR - please explain why you think so. Thanks for cooperation 🤖

Generated by 🚫 dangerJS against cc56bdc

@gibrown
Copy link
Member

gibrown commented Dec 19, 2019

Can we just sync everything that is now in master into instant-search-master? For instance all the php code changes also. Or is that a separate step afterwards?

@bluefuton
Copy link
Contributor Author

Yes - we could also just merge everything back from master into instant-search-master after #14247 ships. Just wanted to make sure these changes didn't get forgotten about.

@jsnmoon
Copy link
Contributor

jsnmoon commented Dec 28, 2019

This PR is no longer necessary; changes from #14247 have already been ported to instant-search-master.

@jsnmoon jsnmoon closed this Dec 28, 2019
@kraftbj kraftbj deleted the instant-search-sync-master-merge-changes branch January 4, 2021 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Search For all things related to Search Touches WP.com Files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants