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

[UA] Move Es deprecations logs to a flyout #215482

Draft
wants to merge 2 commits into
base: 8.x
Choose a base branch
from

Conversation

SoniaSanzV
Copy link
Contributor

Fixes #211496

Summary

This PR introduces some UI changes in the UA step 3: Address API deprecations. Instead of opening the deprecations info in a new window, the info is now displayed in a flyout. Also, a direct button to Discovery has been added in the overview page.

Before

UA step
Image

The page View logs button takes the user to
Image

The filtered Discover view user gets to by clicking on Analyze logs in Discover link
Image

After

UA step
Screenshot 2025-03-21 at 12 47 33
Screenshot 2025-03-21 at 12 47 46
Screenshot 2025-03-21 at 12 47 59
Screenshot 2025-03-21 at 12 48 18
Screenshot 2025-03-21 at 12 48 23

How to test

  • Follow the instructions in https://github.com/elastic/kibana-team/issues/1521. Use any data folder.
  • Go to "Stack Management > Upgrade Assistant"
    • Navigate to "Elasticsearch deprecation issues"
      • Click in the row outside of the issue link to verify that the flyout opens.
    • Navigate to "Kibana deprecation issues"
      • Click in the row outside of the issue link to verify that the flyout opens.

Demo

Video
Screen.Recording.2025-03-21.at.12.52.45.mov

@SoniaSanzV SoniaSanzV added 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 Feature:Upgrade Assistant backport:version Backport to applied version labels v8.18.0 v8.19.0 labels Mar 21, 2025
@SoniaSanzV SoniaSanzV requested a review from a team March 21, 2025 12:26
@SoniaSanzV SoniaSanzV self-assigned this Mar 21, 2025
@SoniaSanzV SoniaSanzV marked this pull request as ready for review March 21, 2025 13:16
@elasticmachine
Copy link
Contributor

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

@SoniaSanzV SoniaSanzV marked this pull request as draft March 21, 2025 13:20
@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!

@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 21, 2025

💔 Build Failed

Failed CI Steps

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
upgradeAssistant 213.1KB 213.6KB +432.0B

Page load bundle

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

id before after diff
upgradeAssistant 23.8KB 23.5KB -242.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
upgradeAssistant 10 11 +1

Total ESLint disabled count

id before after diff
upgradeAssistant 10 11 +1

History

cc @SoniaSanzV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels Feature:Upgrade Assistant 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 v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants