Skip to content

Bring fork up-to-date and fix method-override issue #1

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

Merged
merged 2 commits into from
Jul 17, 2020

Conversation

papanikge
Copy link

This PR is a "hard reset" on the tag v7.1.0 which is our current bundled version
plus
two more commits that fix elastic#924

@papanikge papanikge requested a review from greenonion July 15, 2020 13:53
@papanikge papanikge self-assigned this Jul 15, 2020
@papanikge
Copy link
Author

@greenonion only review the last two commits

Copy link
Member

@greenonion greenonion left a comment

Choose a reason for hiding this comment

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

LGTM!

@papanikge
Copy link
Author

skroutz main branch rebased. moving w/ the merge.

@papanikge papanikge merged commit ae7852e into skroutz Jul 17, 2020
@papanikge papanikge deleted the skroutz-elastic7 branch July 17, 2020 12:19
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.

Elasticsearch::Model overrides class .search method
2 participants