Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ILM] fix remove the unnecessary toast notification #201524

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

viajes7
Copy link
Contributor

@viajes7 viajes7 commented Nov 25, 2024

Summary

Fixes #129647

After fixup:

image

@viajes7 viajes7 requested a review from a team as a code owner November 25, 2024 09:19
@viajes7 viajes7 force-pushed the fix/ilm_toast_error branch 5 times, most recently from 9b60885 to f352650 Compare November 28, 2024 07:36
@viajes7
Copy link
Contributor Author

viajes7 commented Nov 28, 2024

Hi, @ElenaStoeva can you help review this PR? Thanks.

@viajes7 viajes7 force-pushed the fix/ilm_toast_error branch from f352650 to 191e4b5 Compare November 28, 2024 14:06
@ElenaStoeva ElenaStoeva self-requested a review November 28, 2024 14:24
@ElenaStoeva ElenaStoeva added Feature:ILM Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Nov 28, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

@ElenaStoeva
Copy link
Contributor

/ci

@ElenaStoeva
Copy link
Contributor

@elasticmachine run docs-build

Copy link
Contributor

@ElenaStoeva ElenaStoeva left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this @viajes7! Changes lgtm, tested locally.

@ElenaStoeva ElenaStoeva merged commit 9724c46 into elastic:main Nov 28, 2024
16 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 28, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
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 Nov 28, 2024
…202215)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ILM] fix remove the unnecessary toast notification
(#201524)](#201524)

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

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

<!--BACKPORT [{"author":{"name":"Jusheng
Huang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-28T16:58:57Z","message":"[ILM]
fix remove the unnecessary toast notification (#201524)\n\n##
Summary\r\n\r\nFixes #129647 \r\n\r\nAfter fixup:\r\n\r\n<img
width=\"1920\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9e423bae-63c2-4d4e-8a9f-10a922125eb8\">","sha":"9724c46be1257a4287bb313a1cb85407bf0349e3","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:ILM","Team:Kibana
Management","release_note:skip","💝community","v9.0.0","backport:prev-minor"],"title":"[ILM]
fix remove the unnecessary toast
notification","number":201524,"url":"https://github.com/elastic/kibana/pull/201524","mergeCommit":{"message":"[ILM]
fix remove the unnecessary toast notification (#201524)\n\n##
Summary\r\n\r\nFixes #129647 \r\n\r\nAfter fixup:\r\n\r\n<img
width=\"1920\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9e423bae-63c2-4d4e-8a9f-10a922125eb8\">","sha":"9724c46be1257a4287bb313a1cb85407bf0349e3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201524","number":201524,"mergeCommit":{"message":"[ILM]
fix remove the unnecessary toast notification (#201524)\n\n##
Summary\r\n\r\nFixes #129647 \r\n\r\nAfter fixup:\r\n\r\n<img
width=\"1920\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9e423bae-63c2-4d4e-8a9f-10a922125eb8\">","sha":"9724c46be1257a4287bb313a1cb85407bf0349e3"}}]}]
BACKPORT-->

Co-authored-by: Jusheng Huang <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) 💝community Feature:ILM release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ILM- when the ILM form is invalid, no request should be sent hence there should be no toast.
4 participants