Skip to content

[9.2] [ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)#241581

Merged
KodeRad merged 1 commit intoelastic:9.2from
KodeRad:backport/9.2/pr-239883
Nov 3, 2025
Merged

[9.2] [ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)#241581
KodeRad merged 1 commit intoelastic:9.2from
KodeRad:backport/9.2/pr-239883

Conversation

@KodeRad
Copy link
Copy Markdown
Contributor

@KodeRad KodeRad commented Nov 3, 2025

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

…elastic#239883)

## Summary

Implements elastic#236277

This PR:
- Updates the feedback form to use a new short link that points to the
Qualtrics survey instead of Google Forms
- Displays the Feedback button for all job types. Not only hosts
- Fixes React's unique "key" prop warning in console

https://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a

How to test if it displays for hosts:
> To test, it's possible to change the URL of the recognizer page to
create jobs:
> Open the recognizer page for the `Kibana Sample Data Logs` index.
> In the URL change `sample_data_weblogs` to `metrics_ui_hosts`
> The created jobs will not find any anomalies, but they will open in
the anomaly explorer and single metric viewer.

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [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 dbcf492)

# Conflicts:
#	x-pack/platform/plugins/private/translations/translations/de-DE.json
#	x-pack/platform/plugins/private/translations/translations/fr-FR.json
#	x-pack/platform/plugins/private/translations/translations/ja-JP.json
#	x-pack/platform/plugins/private/translations/translations/zh-CN.json
@KodeRad KodeRad added the backport This PR is a backport of another PR label Nov 3, 2025
@KodeRad KodeRad enabled auto-merge (squash) November 3, 2025 10:02
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
ml 2543 2542 -1

Async chunks

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

id before after diff
ml 5.4MB 5.4MB -878.0B

Page load bundle

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

id before after diff
ml 86.3KB 86.1KB -118.0B
@KodeRad KodeRad merged commit dd421f4 into elastic:9.2 Nov 3, 2025
16 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