Skip to content

[9.3] [Discover][Field Formatter] Fix handling of missing values (#251892)#252297

Merged
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-251892
Feb 9, 2026
Merged

[9.3] [Discover][Field Formatter] Fix handling of missing values (#251892)#252297
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-251892

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

@kibanamachine kibanamachine commented Feb 9, 2026

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation
@jughosta
[Discover][Field Formatter] Fix handling of missing values (elastic#2…
14af12a 
…51892)

- Resolves elastic#251846

## Summary

URL and Badge (for boolean type) field formatters had noticeable issues
when field had no data. Some of the other formatters did not use the
same style, so I updated it too.

Also extended the tests.

A regression after elastic#233369

### 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] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [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 deec23d)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 9, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) February 9, 2026 12:30
@kibanamachine kibanamachine mentioned this pull request Feb 9, 2026
Merged
4 tasks
@kibanamachine kibanamachine merged commit 60e4862 into elastic:9.3 Feb 9, 2026
17 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 9, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #30 / Endpoint plugin @ess @serverless @skipInServerlessMKI Endpoint policy response api GET /api/endpoint/policy_response "before all" hook for "should return one policy response for an id"

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
fieldFormats 251 282 +31

Any counts in public APIs

Total count of every any typed public API. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats any for more detailed information.

id before after diff
fieldFormats 6 14 +8

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
fieldFormats 58.3KB 59.8KB +1.5KB
Unknown metric groups

API count

id before after diff
fieldFormats 290 321 +31

cc @jughosta
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

@kibanamachine @elasticmachine @jughosta