Skip to content

Update the test subjects of the feedback buttons#197311

Merged
gbamparop merged 2 commits intoelastic:mainfrom
gbamparop:feedback-buttons-test-subj
Oct 25, 2024
Merged

Update the test subjects of the feedback buttons#197311
gbamparop merged 2 commits intoelastic:mainfrom
gbamparop:feedback-buttons-test-subj

Conversation

@gbamparop
Copy link
Copy Markdown
Contributor

@gbamparop gbamparop commented Oct 22, 2024

Update the test subjects of the feedback buttons.

Closes #196297

@gbamparop gbamparop requested a review from a team as a code owner October 22, 2024 17:38
@gbamparop gbamparop requested a review from a team October 22, 2024 17:38
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Oct 22, 2024
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@gbamparop gbamparop added release_note:skip Skip the PR/issue when compiling release notes v8.16.0 labels Oct 22, 2024
@gbamparop
Copy link
Copy Markdown
Contributor Author

@mykolaharmash do we need to update any dashboards?

@mykolaharmash
Copy link
Copy Markdown
Contributor

mykolaharmash commented Oct 23, 2024

@mykolaharmash do we need to update any dashboards?

We don't because these buttons trigger a custom event, dashboards don't rely on the automatic click event with the subject ID.

Copy link
Copy Markdown
Contributor

@mykolaharmash mykolaharmash left a comment

Choose a reason for hiding this comment

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

@gbamparop
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 24, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: f5ae8f6
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-197311-f5ae8f633499

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
observabilityOnboarding 270.8KB 270.7KB -20.0B

History

@gbamparop gbamparop merged commit 188beda into elastic:main Oct 25, 2024
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.16, 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 25, 2024
Update the test subjects of the feedback buttons.

Closes elastic#196297

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 188beda)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 25, 2024
Update the test subjects of the feedback buttons.

Closes elastic#196297

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 188beda)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 25, 2024
…97763)

# Backport

This will backport the following commits from `main` to `8.16`:
- [Update the test subjects of the feedback buttons
(#197311)](#197311)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Giorgos
Bamparopoulos","email":"georgios.bamparopoulos@elastic.co"},"sourceCommit":{"committedDate":"2024-10-25T07:14:00Z","message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","v8.16.0"],"title":"Update
the test subjects of the feedback
buttons","number":197311,"url":"https://github.com/elastic/kibana/pull/197311","mergeCommit":{"message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197311","number":197311,"mergeCommit":{"message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Giorgos Bamparopoulos <georgios.bamparopoulos@elastic.co>
kibanamachine added a commit that referenced this pull request Oct 25, 2024
…7764)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Update the test subjects of the feedback buttons
(#197311)](#197311)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Giorgos
Bamparopoulos","email":"georgios.bamparopoulos@elastic.co"},"sourceCommit":{"committedDate":"2024-10-25T07:14:00Z","message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","v8.16.0"],"title":"Update
the test subjects of the feedback
buttons","number":197311,"url":"https://github.com/elastic/kibana/pull/197311","mergeCommit":{"message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/197311","number":197311,"mergeCommit":{"message":"Update
the test subjects of the feedback buttons (#197311)\n\nUpdate the test
subjects of the feedback buttons.\r\n\r\nCloses
https://github.com/elastic/kibana/issues/196297\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"188beda849ae886b164c9ccd3d5f7d2edf35479c"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Giorgos Bamparopoulos <georgios.bamparopoulos@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes v8.16.0 v8.17.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Observability Onboarding] Change test-subj id on the feedback buttons

4 participants