Skip to content

Fix broken translation string#268460

Merged
sphilipse merged 2 commits into
elastic:mainfrom
sphilipse:harness/fix-broken-translation-string
May 8, 2026
Merged

Fix broken translation string#268460
sphilipse merged 2 commits into
elastic:mainfrom
sphilipse:harness/fix-broken-translation-string

Conversation

@sphilipse
Copy link
Copy Markdown
Member

@sphilipse sphilipse commented May 8, 2026

Summary

This is a quick fix for a broken translating string that was spamming the browser console.

@sphilipse sphilipse added the release_note:skip Skip the PR/issue when compiling release notes label May 8, 2026
@sphilipse sphilipse requested a review from a team as a code owner May 8, 2026 14:16
Copy link
Copy Markdown
Contributor

@lorenabalan lorenabalan left a comment

Choose a reason for hiding this comment

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

Comment thread src/platform/packages/shared/kbn-connector-specs/src/specs/snowflake/snowflake.ts Outdated
…wflake/snowflake.ts

Co-authored-by: Lorena Bălan <lorena.balan@elastic.co>
@sphilipse sphilipse enabled auto-merge (squash) May 8, 2026 14:22
@mattnowzari
Copy link
Copy Markdown
Contributor

@lorenabalan @sphilipse my PR has additional fixes this one doesn't have - should we defer to mine? I'm good either way, that one also aligns on some variances in how the strings are formatted across several connectors.

Sorry for the delay, timing has been a bit of a juggle - I promise I'll get that one shaped up today!

@sphilipse
Copy link
Copy Markdown
Member Author

@lorenabalan @sphilipse my PR has additional fixes this one doesn't have - should we defer to mine? I'm good either way, that one also aligns on some variances in how the strings are formatted across several connectors.

Sorry for the delay, timing has been a bit of a juggle - I promise I'll get that one shaped up today!

Let's merge whatever gets there first. It's a trivial merge conflict to resolve if it results in one. I'd want to prioritize making sure we silence the browser console noise. The rest is nice to have but less urgent.

@kibanamachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout Lane #9 - stateful-classic / default / local-stateful-classic - APM integration not installed but setup completed - Admin user
  • [job] [logs] Scout Lane #9 - stateful-classic / default / local-stateful-classic - Collector integration is not installed - collector integration missing
  • [job] [logs] Scout Lane #9 - stateful-classic / default / local-stateful-classic - Collector integration is not installed - Symbolizer integration is not installed
  • [job] [logs] Scout Lane #9 - stateful-classic / default / local-stateful-classic - Profiling is not setup and no data is loaded - Admin users
  • [job] [logs] Scout Lane #9 - stateful-classic / default / local-stateful-classic - Profiling is not setup and no data is loaded - Viewer users

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
stackConnectors 1.8MB 1.8MB +51.0B
workflowsManagement 2.3MB 2.3MB +51.0B
total +102.0B

@sphilipse sphilipse merged commit 2d58e1b into elastic:main May 8, 2026
32 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.4

https://github.com/elastic/kibana/actions/runs/25564378574

@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
9.4 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.4:
- [One Workflow] Unskip Workflow editor: validation performance tests (#268149)
- [AI Infra] Increase GenAI Settings Scout navigation outer timeout (#268392)
- chore(axios,security-solution): remove axios from telemetry/role scripts (#267944)
- [scout] fix cleanup in SO managerment find test (#268349)

Manual backport

To create the backport manually run:

node scripts/backport --pr 268460

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 12, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 268460 locally
cc: @sphilipse

1 similar comment
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 268460 locally
cc: @sphilipse

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

Labels

backport missing Added to PRs automatically when the are determined to be missing a backport. backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Search v9.4.0 v9.4.1 v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants