Skip to content

[RAM] Text not localized on rules managament table#166700

Merged
guskovaue merged 4 commits intoelastic:mainfrom
guskovaue:RAM-165552-text-not-localized-on-mngt-table
Sep 27, 2023
Merged

[RAM] Text not localized on rules managament table#166700
guskovaue merged 4 commits intoelastic:mainfrom
guskovaue:RAM-165552-text-not-localized-on-mngt-table

Conversation

@guskovaue
Copy link
Copy Markdown
Contributor

@guskovaue guskovaue commented Sep 19, 2023

Summary

Fixed issue: #165552 for all existing locale.

Looked before:
Screenshot 2023-09-19 at 12 24 08

Now looks like this:
Screenshot 2023-09-19 at 12 19 49

@guskovaue guskovaue added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.11.0 labels Sep 19, 2023
@guskovaue guskovaue self-assigned this Sep 19, 2023
@XavierM XavierM requested a review from umbopepato September 19, 2023 15:54
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You do not need to add these translation because at each release, we have a process to rebuild all these translations. So, It is ok to remove it, you only get error when you have duplicate key.

Copy link
Copy Markdown
Contributor Author

@guskovaue guskovaue Sep 21, 2023

Choose a reason for hiding this comment

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

@jcger We just had an exact translations we need for that, so I decided to save translators time. And also for testing (to see that we actually have everuthing translated).
Do you think it's safer to delete them for now?

Copy link
Copy Markdown
Contributor

@jcger jcger Sep 21, 2023

Choose a reason for hiding this comment

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

Didn't knew you speak French, Japanese and Chinese! :P

I would say yes, you should remove them but I'm not 100% sure. My comment was copied from another comment I received in a PR about a year ago cc @XavierM

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jcger I have a lot of hidden talents :-P.

I've fixed it. Probably you are right. It's safer. And I've alredy tested it with translations.

@guskovaue guskovaue force-pushed the RAM-165552-text-not-localized-on-mngt-table branch from d61960c to 3dcd171 Compare September 26, 2023 13:45
@guskovaue
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
triggersActionsUi 1.4MB 1.4MB +380.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @guskovaue

@guskovaue guskovaue marked this pull request as ready for review September 26, 2023 16:25
@guskovaue guskovaue requested a review from a team as a code owner September 26, 2023 16:25
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@guskovaue guskovaue merged commit 255bf6e into elastic:main Sep 27, 2023
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Sep 27, 2023
@guskovaue guskovaue linked an issue Sep 27, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.11.0

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

Text not localized on rule management table

5 participants