Skip to content

[9.1] add loading state install/uninstall product doc (#229579)#230616

Merged
arturoliduena merged 1 commit intoelastic:9.1from
arturoliduena:backport/9.1/pr-229579
Aug 5, 2025
Merged

[9.1] add loading state install/uninstall product doc (#229579)#230616
arturoliduena merged 1 commit intoelastic:9.1from
arturoliduena:backport/9.1/pr-229579

Conversation

@arturoliduena
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

Closes [elastic#229460](elastic#229460)
## Summary

When a user clicks the "Install" or "Uninstall" button for product
documentation, add visual feedback in the UI (a spinner) to indicate
that an action is in progress.

### Steps to Reproduce
Navigate Observability AI Assistant settings page

see that Elastic documentation is not yet installed.

Click the Install button.

Observe that the button shows no loading state.

(Similarly, for an installed package, click Uninstall and observe the
same lack of feedback).

Expected Behavior
After clicking "Install" or "Uninstall", the button should immediately
enter a loading state.

https://github.com/user-attachments/assets/9e9f8fef-bf18-4a6a-96af-c514dae4ffdf

https://github.com/user-attachments/assets/1c36d617-0def-4a6d-af04-bb9ef52c624f

https://github.com/user-attachments/assets/8053cf33-cc69-4d1d-8779-23a361266576

### 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)

(cherry picked from commit 8e9fabe)

# Conflicts:
#	x-pack/platform/plugins/private/observability_ai_assistant_management/public/hooks/use_get_product_doc.ts
@arturoliduena arturoliduena added the backport This PR is a backport of another PR label Aug 5, 2025
@arturoliduena arturoliduena enabled auto-merge (squash) August 5, 2025 15:19
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Aug 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 5, 2025

🤖 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!)

@elasticmachine
Copy link
Contributor

elasticmachine commented Aug 5, 2025

💚 Build Succeeded

  • Buildkite Build
  • Commit: b77693d
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-230616-b77693de6b8a

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
observabilityAiAssistantManagement 97.7KB 98.5KB +816.0B

Page load bundle

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

id before after diff
observabilityAiAssistantManagement 5.7KB 5.7KB +2.0B

History

@arturoliduena arturoliduena merged commit 19c9b87 into elastic:9.1 Aug 5, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR ci:project-deploy-observability Create an Observability project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants