[ML] Anomaly Detection: Adds feedback button to the Anomaly Explorer and Single Metric Viewer#239883
Merged
KodeRad merged 13 commits intoelastic:mainfrom Oct 27, 2025
Merged
Conversation
KodeRad
commented
Oct 21, 2025
| */ | ||
|
|
||
| import type { ReactElement } from 'react'; | ||
| import React from 'react'; |
Contributor
Author
There was a problem hiding this comment.
This file is updated accordingly to the observability one: x-pack/solutions/observability/plugins/observability_shared/public/components/feature_feedback_button/feature_feedback_button.tsx
KodeRad
commented
Oct 22, 2025
| <MlPageHeader | ||
| leftSideItems={[ | ||
| <AnomalyResultsViewSelector | ||
| key="anomaly-results-view-selector" |
Contributor
Author
There was a problem hiding this comment.
This key was added to prevent React's unique "key" prop warning in console
KodeRad
commented
Oct 22, 2025
| setShowButton( | ||
| resp.some((job) => MATCHED_CREATED_BY_TAGS.includes(job.custom_settings?.created_by)) | ||
| ); | ||
| setShowButton(resp.length > 0); |
Contributor
Author
There was a problem hiding this comment.
We allow all types of jobs. Not only hosts. Here I am checking if there are any jobs whatsoever.
Member
There was a problem hiding this comment.
Originally we loaded the job just to get access to the created_by setting so we could ensure it was ml-module-metrics-ui-hosts, as in, we only showed this button if the job had been created via the metrics_ui_hosts module which is used by the observability solution.
Now that we are no longer making this check, we do not need to load the jobs.
We could add a check to not show the button if the jobIds prop is empty
Member
There was a problem hiding this comment.
Looking at what's inside FeedBackButton and FeatureFeedbackButton, if we're getting rid of the created_by check and not loading the jobs, I'm thinking we could merge these two components.
I believe they were originally created as separate components so that FeatureFeedbackButton could be generic and reused. And we were following the lead of the observability feedback button.
I think this is now not needed and we will only have one AD feedback button.
Contributor
Author
There was a problem hiding this comment.
Thank you for catching this 👍 Fixed in caaa972
KodeRad
added
:ml
Feature:Anomaly Detection
KodeRad
changed the title
KodeRad
added
release_note:enhancement
backport:all-open
Contributor
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#9608[✅] x-pack/platform/test/functional/apps/ml/anomaly_detection_result_views/config.ts: 25/25 tests passed. |
Contributor
|
Pinging @elastic/ml-ui (:ml) |
20 tasks
| nodeType?: NodeType; | ||
| }) => { | ||
| const deploymentType = | ||
| isCloudEnv !== undefined || isServerlessEnv !== undefined |
Member
There was a problem hiding this comment.
Could these undefined checks be moved into getDeploymentType?
Contributor
Author
There was a problem hiding this comment.
This is the exact copy of the observability feature_feedback_button.tsx here Does it make more sense to customize it for our needs?
.../plugins/shared/ml/public/application/components/feedback_button/feature_feedback_button.tsx
Outdated
Show resolved
Hide resolved
peteharverson
approved these changes
Oct 24, 2025
Contributor
peteharverson
left a comment
peteharverson
left a comment
There was a problem hiding this comment.
Latest changes LGTM
Contributor
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
Page load bundle
History
cc @KodeRad |
Contributor
|
Starting backport for target branches: 8.19, 9.1, 9.2 |
Contributor
💔 All backports failed
Manual backportTo create the backport manually run: Questions ?Please refer to the Backport tool documentation |
KodeRad
added
backport:skip
Contributor
Author
|
Skipping the backport as for now. We will decide if it needs backporting and we will re-visit |
peteharverson
changed the title
Contributor
Author
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
KodeRad
added a commit
to KodeRad/kibana
that referenced
this pull request
Nov 3, 2025
…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
added a commit
to KodeRad/kibana
that referenced
this pull request
Nov 3, 2025
…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 # x-pack/platform/plugins/shared/ml/public/application/components/feedback_button/feature_feedback_button.tsx # x-pack/platform/plugins/shared/ml/public/application/timeseriesexplorer/timeseriesexplorer_page.tsx
Contributor
Author
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
KodeRad
added a commit
to KodeRad/kibana
that referenced
this pull request
Nov 3, 2025
…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 # x-pack/platform/plugins/shared/ml/public/application/components/feedback_button/feature_feedback_button.tsx # x-pack/platform/plugins/shared/ml/public/application/timeseriesexplorer/timeseriesexplorer_page.tsx
KodeRad
added a commit
that referenced
this pull request
Nov 3, 2025
…update (#239883) (#241583) # Backport This will backport the following commits from `main` to `9.1`: - [[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)](#239883) <!--- Backport version: 10.1.0 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Konrad Krasocki","email":"104936644+KodeRad@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-10-27T10:35:38Z","message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement",":ml","Feature:Anomaly Detection","backport:skip","Team:ML","v9.3.0"],"title":"[ML] Anomaly Detection: Adds feedback button to the Anomaly Explorer and Single Metric Viewer","number":239883,"url":"https://github.com/elastic/kibana/pull/239883","mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239883","number":239883,"mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}},{"url":"https://github.com/elastic/kibana/pull/241581","number":241581,"branch":"9.2","state":"OPEN"}]}] BACKPORT--> --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
KodeRad
added a commit
that referenced
this pull request
Nov 3, 2025
…update (#239883) (#241581) # Backport This will backport the following commits from `main` to `9.2`: - [[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)](#239883) <!--- Backport version: 10.1.0 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Konrad Krasocki","email":"104936644+KodeRad@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-10-27T10:35:38Z","message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement",":ml","Feature:Anomaly Detection","backport:skip","Team:ML","v9.3.0"],"title":"[ML] Anomaly Detection: Adds feedback button to the Anomaly Explorer and Single Metric Viewer","number":239883,"url":"https://github.com/elastic/kibana/pull/239883","mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239883","number":239883,"mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}}]}] BACKPORT-->
KodeRad
added a commit
that referenced
this pull request
Nov 3, 2025
… update (#239883) (#241586) # Backport This will backport the following commits from `main` to `8.19`: - [[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)](#239883) <!--- Backport version: 10.1.0 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Konrad Krasocki","email":"104936644+KodeRad@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-10-27T10:35:38Z","message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement",":ml","Feature:Anomaly Detection","backport:skip","Team:ML","v9.3.0"],"title":"[ML] Anomaly Detection: Adds feedback button to the Anomaly Explorer and Single Metric Viewer","number":239883,"url":"https://github.com/elastic/kibana/pull/239883","mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/239883","number":239883,"mergeCommit":{"message":"[ML] Anomaly Explorer and Single Metric Viewer feedback button update (#239883)\n\n## Summary\n\nImplements #236277\n\nThis PR:\n- Updates the feedback form to use a new short link that points to the\nQualtrics survey instead of Google Forms\n- Displays the Feedback button for all job types. Not only hosts\n- Fixes React's unique \"key\" prop warning in console\n\n\n\nhttps://github.com/user-attachments/assets/fcd7c8b0-e95e-462f-964f-dfebe3f6700a\n\nHow to test if it displays for hosts:\n> To test, it's possible to change the URL of the recognizer page to\ncreate jobs:\n> Open the recognizer page for the `Kibana Sample Data Logs` index. \n> In the URL change `sample_data_weblogs` to `metrics_ui_hosts` \n> The created jobs will not find any anomalies, but they will open in\nthe anomaly explorer and single metric viewer.\n\n\n### Checklist\n\nCheck the PR satisfies following conditions. \n\nReviewers should verify this PR satisfies this list as well.\n\n- [x] Any text added follows [EUI's writing\nguidelines](https://elastic.github.io/eui/#/guidelines/writing), uses\nsentence case text and includes [i18n\nsupport](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)\n- [x] Review the [backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand apply applicable `backport:*` labels.","sha":"dbcf492bc6b78f4fe550bd6cef714819d7e7794e"}},{"url":"https://github.com/elastic/kibana/pull/241581","number":241581,"branch":"9.2","state":"OPEN"}]}] BACKPORT-->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements #236277
This PR:
feedback-button.mp4
How to test if it displays for hosts:
Checklist
Check the PR satisfies following conditions.
Reviewers should verify this PR satisfies this list as well.
release_note:*label is applied per the guidelinesbackport:*labels.