Skip to content

Conversation

@Nox911
Copy link
Contributor

@Nox911 Nox911 commented Jan 23, 2019

Translate x-pack -> plugins -> kuery_autocomplete

Guidline
Readme

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@Nox911 Nox911 requested review from pavel06081991 and removed request for maryia-lapata January 24, 2019 07:44
Copy link
Contributor

@pavel06081991 pavel06081991 left a comment

Choose a reason for hiding this comment

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

LGTM. Tested locally, works good.

@Nox911
Copy link
Contributor Author

Nox911 commented Jan 25, 2019

Fixed description and replaced concat to backticks in values(Aleh Zasypkin asked me to do it).

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Code LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@Nox911 Nox911 merged commit 905ea52 into elastic:master Jan 29, 2019
@Nox911 Nox911 deleted the feature/translate-kuery_autocomplete branch January 29, 2019 15:41
Nox911 added a commit to Nox911/kibana that referenced this pull request Jan 29, 2019
* Translate kuery_autocomplete

* Resolve review comments
Nox911 added a commit that referenced this pull request Jan 29, 2019
* Translate kuery_autocomplete

* Resolve review comments
@Nox911
Copy link
Contributor Author

Nox911 commented Jan 29, 2019

6.x/6.7.0: 73567a2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants