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

translate tutorials(kafka_logs, kafka_metrics, kibana_metrics, kubern… #22122

Merged

Conversation

pavel06081991
Copy link
Contributor

#21957
translate tutorials(kafka_logs, kafka_metrics, kibana_metrics, kubernetes_metrics, logstash_logs, logstash_metrics, memcached_metrics)

…etes_metrics, logstash_logs, logstash_metrics, memcached_metrics)
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@pavel06081991 pavel06081991 added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Project:i18n :Sharing and removed Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc labels Aug 20, 2018
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@yankouskia yankouskia requested a review from tsullivan August 22, 2018 12:04
Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

Found a duplicated string ID.

Is it possible to have the framework throw an error if this happens?

shortDescription: i18n.translate('kbn.server.tutorials.memcachedMetrics.shortDescription', {
defaultMessage: 'Fetch internal metrics from the Memcached server.',
}),
longDescription: i18n.translate('kbn.server.tutorials.memcachedMetrics.shortDescription', {
Copy link
Member

Choose a reason for hiding this comment

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

kbn.server.tutorials.memcachedMetrics.longDescription

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed. thanks!

@pavel06081991
Copy link
Contributor Author

@tsullivan I believe we do not need to throw errors. Soon we will have such validations like duplicating ids in our CI pipeline. So we will not be able to merge pull request if we have duplicating ids

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

LGTM

@pavel06081991 pavel06081991 merged commit a445e2d into elastic:master Aug 24, 2018
@pavel06081991 pavel06081991 deleted the feature/tutorial-translations-part3 branch August 24, 2018 08:48
pavel06081991 added a commit to pavel06081991/kibana that referenced this pull request Aug 24, 2018
elastic#22122)

translate tutorials(kafka_logs, kafka_metrics, kibana_metrics, kubernetes_metrics, logstash_logs, logstash_metrics, memcached_metrics)
# Conflicts:
#	src/core_plugins/kibana/server/tutorials/logstash_logs/index.js
pavel06081991 added a commit that referenced this pull request Aug 24, 2018
#22122) (#22351)

translate tutorials(kafka_logs, kafka_metrics, kibana_metrics, kubernetes_metrics, logstash_logs, logstash_metrics, memcached_metrics)
# Conflicts:
#	src/core_plugins/kibana/server/tutorials/logstash_logs/index.js
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