Skip to content

Feature/translate canvas public lib#23752

Closed
Nox911 wants to merge 23 commits intoelastic:masterfrom
Nox911:feature/translate-canvas-public-lib
Closed

Feature/translate canvas public lib#23752
Nox911 wants to merge 23 commits intoelastic:masterfrom
Nox911:feature/translate-canvas-public-lib

Conversation

@Nox911
Copy link
Copy Markdown
Contributor

@Nox911 Nox911 commented Oct 3, 2018

translate canvas public lib

#23222

@Nox911 Nox911 requested a review from pavel06081991 October 3, 2018 12:52
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
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 but please fix comments

@rashidkpc rashidkpc added the Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// label Oct 3, 2018
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Contributor

@clintandrewhall clintandrewhall left a comment

Choose a reason for hiding this comment

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

My comments are below.

Copy link
Copy Markdown
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.

please check all places where you disable eslint rules. When you disable some line you should add name of rule you disable

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@pavel06081991
Copy link
Copy Markdown
Contributor

@clintandrewhall please review one more time

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

Labels

Project:i18n Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t//

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants