Skip to content

[8.19] [Lens] Forward secondary prefix correctly when state value is an empty string (None option) (#228183)#228423

Merged
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-228183
Jul 22, 2025
Merged

[8.19] [Lens] Forward secondary prefix correctly when state value is an empty string (None option) (#228183)#228423
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-228183

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…y string (`None` option) (elastic#228183)

Closes elastic#227281

## Summary

When going from state to expression, the secondaryPrefix value wasn't
forwarded correctly when the secondaryPrefix state value was an empty
string (the empty string is set in the state when the Prefix "None"
option is selected in settings). It was forwarded as `undefined`,
causing showing the column name as the prefix on:

https://github.com/elastic/kibana/blob/2928bdf1cb5f8fd648c6a46a52ae3f96b4b01a70/src/platform/plugins/shared/chart_expressions/expression_metric/public/components/secondary_metric.tsx#L255

#### After:

https://github.com/user-attachments/assets/1a104f9e-bf78-4acc-aca5-a3866c80d03f

### Checklist
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit d1846d3)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jul 17, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) July 17, 2025 10:56
@elasticmachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

Metrics [docs]

‼️ ERROR: no builds found for mergeBase sha [52788c4]

cc @mariairiartef

@kibanamachine kibanamachine merged commit 63737f2 into elastic:8.19 Jul 22, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

3 participants