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

Moving operators boolean feature to ElasticPressLabs #7

Merged
merged 12 commits into from
Sep 1, 2021

Conversation

Rahmon
Copy link
Contributor

@Rahmon Rahmon commented Aug 20, 2021

Description of the Change

This change adds the Boolean Search Operators Feature.

Ref: 10up/ElasticPress#2293

Alternate Designs

Benefits

Possible Drawbacks

Verification Process

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

@Rahmon Rahmon added the enhancement New feature or request label Aug 20, 2021
@Rahmon Rahmon added this to the 1.2.0 milestone Aug 20, 2021
@Rahmon Rahmon self-assigned this Aug 20, 2021
@Rahmon Rahmon marked this pull request as ready for review August 27, 2021 18:29
@Rahmon Rahmon assigned felipeelia and unassigned Rahmon Aug 27, 2021
@Rahmon Rahmon requested review from felipeelia and moraleida August 27, 2021 18:32
@moraleida
Copy link
Contributor

LGTM, thank you for adding the tests @Rahmon !

@felipeelia felipeelia removed their assignment Aug 30, 2021
@Rahmon Rahmon self-assigned this Aug 30, 2021
@Rahmon Rahmon assigned felipeelia and unassigned Rahmon Aug 30, 2021
@Rahmon Rahmon requested a review from felipeelia August 30, 2021 23:05
@felipeelia felipeelia removed their assignment Aug 31, 2021
@Rahmon Rahmon merged commit 2d0553f into develop Sep 1, 2021
@Rahmon Rahmon deleted the chore/move-boolean-operators-from-EP branch September 1, 2021 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants