-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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 Reporting plugin #22757
Translations for Reporting plugin #22757
Conversation
# Conflicts: # .i18nrc.json
Generate {{ exportConfig.exportType.name }} | ||
<span i18n-id="xpack.reporting.exportConfig.generateButtonLabel" | ||
i18n-default-message="Generate {name}" | ||
i18n-values="{ name: exportConfig.exportType.name }"></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting and indentation looks messed up here. Should be like:
<span
i18n-id="xpack.reporting.exportConfig.generateButtonLabel"
i18n-default-message="Generate {name}"
i18n-values="{ name: exportConfig.exportType.name }"
>
</span>
<h2 class="kuiLocalDropdownTitle"> | ||
Reporting | ||
</h2> | ||
<h2 class="kuiLocalDropdownTitle" i18n-id="xpack.reporting.exportConfig.reportingTitle" i18n-default-message="Reporting"></h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's break this line up per attribute, for consistency
Please save your work before generating a report. | ||
<div ng-show="exportConfig.isDirty()" data-test-subj="unsavedChangesReportingWarning" | ||
i18n-id="xpack.reporting.exportConfig.saveWorkDescription" | ||
i18n-default-message="Please save your work before generating a report."> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div
ng-show="exportConfig.isDirty()"
data-test-subj="unsavedChangesReportingWarning"
i18n-id="xpack.reporting.exportConfig.saveWorkDescription"
i18n-default-message="Please save your work before generating a report."
>
</div>
@@ -56,15 +56,16 @@ module.directive('exportConfig', ($rootScope, reportingDocumentControl, reportin | |||
$scope.updateUrl = (options) => { | |||
return $scope.getRelativePath(options) | |||
.then(relativePath => { | |||
$scope.exportConfig.absoluteUrl = createAbsoluteUrl(relativePath); | |||
$scope.exportConfig.absoluteUrl = createAbsoluteUrl(relativePath) | |||
|| i18n('xpack.reporting.exportConfig.generationUrl.loadingPlaceholder', { defaultMessage: 'Loading...' }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This moved code looks like it needs some manual testing. I'm not super familiar with this file. Do you have testing steps for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feedback is mostly requeststo keep formatting and indentation per our coding style
Blocked by #22441 |
💔 Build Failed |
💔 Build Failed |
Hi, can you look into the merge conflict? |
💔 Build Failed |
💔 Build Failed |
Hi, looks like there's one small merge conflict on this. |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
Here was a broken functional test
This was caused by |
* Translate Reporting * Use i18n core * Translate Reporting * Translate Reporting * Refactoring * Add reporting to i18n config * Fix tooltip translation * Apply formatting for html * Translate label in TS files. * Reduce tag number * Rename ids * Update snapshot * Revert changes for removing span * Add types * Translate statuses * update translation of Reporting plugin * fix variable name * fix constants names * update reporting translation * fix copyright * Fix types/angular packages incompatibility * Update yarn.lock * Fix duplicated import * Fix CI * Small adjustments
6.x/6.6.0: #26569 |
* Translate Reporting * Use i18n core * Translate Reporting * Translate Reporting * Refactoring * Add reporting to i18n config * Fix tooltip translation * Apply formatting for html * Translate label in TS files. * Reduce tag number * Rename ids * Update snapshot * Revert changes for removing span * Add types * Translate statuses * update translation of Reporting plugin * fix variable name * fix constants names * update reporting translation * fix copyright * Fix types/angular packages incompatibility * Update yarn.lock * Fix duplicated import * Fix CI * Small adjustments
Apply translations for Reporting plugin
Fixes #22647