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

Translations for Visual Builder (part 1) #23988

Merged

Conversation

maryia-lapata
Copy link
Contributor

@maryia-lapata maryia-lapata commented Oct 12, 2018

Translations for Aggregations in Visual Builder

Issue #23257

@maryia-lapata maryia-lapata added Project:i18n Feature:TSVB TSVB (Time Series Visual Builder) labels Oct 12, 2018
@maryia-lapata maryia-lapata self-assigned this Oct 12, 2018
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. please fix some small comments

src/core_plugins/metrics/public/kbn_vis_types/index.js Outdated Show resolved Hide resolved
src/core_plugins/metrics/public/kbn_vis_types/index.js Outdated Show resolved Hide resolved
@maryia-lapata maryia-lapata requested a review from timroes October 19, 2018 10:29
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@Nox911
Copy link
Contributor

Nox911 commented Nov 21, 2018

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@Nox911 Nox911 requested a review from timroes November 22, 2018 10:49
@timroes
Copy link
Contributor

timroes commented Nov 22, 2018

Could you please resolve the conflicts, then I can give this a final review.

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, approve assuming a green CI

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@Nox911 Nox911 merged commit 6b89748 into elastic:master Nov 26, 2018
@Nox911 Nox911 removed the review label Nov 26, 2018
@maryia-lapata maryia-lapata deleted the feature/translations/visual-builder branch November 26, 2018 10:48
maryia-lapata added a commit to maryia-lapata/kibana that referenced this pull request Nov 26, 2018
* Translations for Aggregations in Visual Builder

* Refactoring

* Translate agg_lookup

* Update ids

* Add translation

* Replace namespace metrics to tsvb

* Change order in i18nrc file

* Fix issues

* Fix issues

* Fix label calculating

* Fix minor issue
Nox911 pushed a commit that referenced this pull request Nov 26, 2018
* Translations for Aggregations in Visual Builder

* Refactoring

* Translate agg_lookup

* Update ids

* Add translation

* Replace namespace metrics to tsvb

* Change order in i18nrc file

* Fix issues

* Fix issues

* Fix label calculating

* Fix minor issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported Feature:TSVB TSVB (Time Series Visual Builder) Project:i18n Team:Visualizations Visualization editors, elastic-charts and infrastructure v6.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants