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

[Index Management] Fix ILM policy link trigger an SPA friendly navigation #199252

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

viajes7
Copy link
Contributor

@viajes7 viajes7 commented Nov 7, 2024

Summary

Close #87876

Use core.application.navigateToUrl navigate to given URL in a SPA friendly.

2024-11-07.11.24.15.mov

@viajes7 viajes7 requested a review from a team as a code owner November 7, 2024 03:25
@viajes7 viajes7 force-pushed the fix/ilm-policy-route branch 2 times, most recently from f17a573 to 2337a22 Compare November 9, 2024 00:27
@viajes7
Copy link
Contributor Author

viajes7 commented Nov 9, 2024

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

@viajes7
Copy link
Contributor Author

viajes7 commented Nov 12, 2024

@elasticmachine merge upstream

@viajes7 viajes7 force-pushed the fix/ilm-policy-route branch 5 times, most recently from 6edf527 to 7b91cd3 Compare November 20, 2024 05:47
@ElenaStoeva ElenaStoeva self-requested a review November 20, 2024 10:16
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! Changes lgtm, tested locally.

@ElenaStoeva ElenaStoeva added Feature:Index Management Index and index templates UI 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 20, 2024
@elasticmachine
Copy link
Contributor

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

@ElenaStoeva
Copy link
Contributor

/ci

@ElenaStoeva
Copy link
Contributor

Hey @viajes7, there is one failing Jest test in ci, which I think is caused by these changes - could you please fix it?

@viajes7
Copy link
Contributor Author

viajes7 commented Nov 21, 2024

Hey @viajes7, there is one failing Jest test in ci, which I think is caused by these changes - could you please fix it?

Sure, test case fixed

@viajes7 viajes7 force-pushed the fix/ilm-policy-route branch from 7b91cd3 to dcca7d3 Compare November 21, 2024 08:31
@ElenaStoeva
Copy link
Contributor

/ci

@ElenaStoeva
Copy link
Contributor

Sure, test case fixed

It looks like the test is still failing - also, I don't see any test changes in this PR.

@viajes7 viajes7 force-pushed the fix/ilm-policy-route branch from e654383 to 1f0b09b Compare November 21, 2024 12:50
@viajes7
Copy link
Contributor Author

viajes7 commented Nov 21, 2024

It looks like the test is still failing - also, I don't see any test changes in this PR.

Hi, @ElenaStoeva Sorry about that. I didn't notice that the local push failed and I have pushed it again.

@ElenaStoeva
Copy link
Contributor

/ci

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #17 / ConnectorSelector should set the selected connector to none if the connector is not available

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
indexManagement 698.5KB 698.8KB +259.0B

History

@ElenaStoeva
Copy link
Contributor

@elasticmachine run docs-build

@ElenaStoeva ElenaStoeva merged commit fa100b4 into elastic:main Nov 21, 2024
23 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 21, 2024
…tion (elastic#199252)

## Summary

Close elastic#87876

Use `core.application.navigateToUrl` navigate to given URL in a SPA
friendly.

https://github.com/user-attachments/assets/1918eb3d-fbec-46d9-8eae-b4a26ebd36a2
(cherry picked from commit fa100b4)
@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 21, 2024
…navigation (#199252) (#201271)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Index Management] Fix ILM policy link trigger an SPA friendly
navigation (#199252)](#199252)

<!--- 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-21T18:35:59Z","message":"[Index
Management] Fix ILM policy link trigger an SPA friendly navigation
(#199252)\n\n## Summary\r\n\r\nClose #87876 \r\n\r\nUse
`core.application.navigateToUrl` navigate to given URL in a
SPA\r\nfriendly.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/1918eb3d-fbec-46d9-8eae-b4a26ebd36a2","sha":"fa100b48191d08fde48894ff88dd49687ef4d524","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Index
Management","Team:Kibana
Management","release_note:skip","💝community","v9.0.0","backport:prev-minor"],"title":"[Index
Management] Fix ILM policy link trigger an SPA friendly
navigation","number":199252,"url":"https://github.com/elastic/kibana/pull/199252","mergeCommit":{"message":"[Index
Management] Fix ILM policy link trigger an SPA friendly navigation
(#199252)\n\n## Summary\r\n\r\nClose #87876 \r\n\r\nUse
`core.application.navigateToUrl` navigate to given URL in a
SPA\r\nfriendly.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/1918eb3d-fbec-46d9-8eae-b4a26ebd36a2","sha":"fa100b48191d08fde48894ff88dd49687ef4d524"}},"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/199252","number":199252,"mergeCommit":{"message":"[Index
Management] Fix ILM policy link trigger an SPA friendly navigation
(#199252)\n\n## Summary\r\n\r\nClose #87876 \r\n\r\nUse
`core.application.navigateToUrl` navigate to given URL in a
SPA\r\nfriendly.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/1918eb3d-fbec-46d9-8eae-b4a26ebd36a2","sha":"fa100b48191d08fde48894ff88dd49687ef4d524"}}]}]
BACKPORT-->

Co-authored-by: Jusheng Huang <[email protected]>
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
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:Index Management Index and index templates UI 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.

[Index Management] ILM link in data streams details flyout should not reload the app
4 participants