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

[unification] Updates to AI Assistant Calls to Action #216105

Merged
merged 4 commits into from
Mar 27, 2025

Conversation

clintandrewhall
Copy link
Contributor

@clintandrewhall clintandrewhall commented Mar 26, 2025

Summary

This PR addresses feedback and includes a number of changes:

  • allow a CTA to be offset by the height of the AssistantBeacon when centered vertically, (e.g. in a tall FlexItem).
  • add data-test-subj prop to panels for consumers writing tests.
  • updates to copy, documentation, tests, storybook.

@clintandrewhall clintandrewhall added review release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) Feature:Security Assistant Security Assistant Team:Obs AI Assistant Observability AI Assistant v9.1.0 labels Mar 26, 2025
@clintandrewhall clintandrewhall requested a review from a team as a code owner March 26, 2025 20:22
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

New copy lgtm, thanks!

clintandrewhall and others added 3 commits March 27, 2025 10:48

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…install_knowledge_base/install_knowledge_base.translations.ts

Co-authored-by: florent-leborgne <[email protected]>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@clintandrewhall clintandrewhall enabled auto-merge (squash) March 27, 2025 15:15
@clintandrewhall clintandrewhall merged commit c4fa63f into elastic:main Mar 27, 2025
10 checks passed
@clintandrewhall clintandrewhall deleted the ai/calls-to-action branch March 27, 2025 17:11
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.0

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/ai-assistant-cta 6 7 +1
@kbn/ai-assistant-icon 3 5 +2
total +3

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observabilityAiAssistantManagement 107.2KB 107.1KB -76.0B
searchAssistant 159.0KB 158.9KB -80.0B
securitySolution 8.9MB 8.9MB -80.0B
securitySolutionEss 36.1KB 36.0KB -78.0B
securitySolutionServerless 101.0KB 101.0KB -78.0B
total -392.0B

Page load bundle

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

id before after diff
observabilityAIAssistantApp 15.8KB 15.7KB -80.0B
Unknown metric groups

API count

id before after diff
@kbn/ai-assistant-cta 28 37 +9
@kbn/ai-assistant-icon 11 13 +2
total +11

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 27, 2025
## Summary

This PR addresses feedback and includes a number of changes:

- allow a CTA to be offset by the height of the `AssistantBeacon` when
centered vertically, (e.g. in a tall `FlexItem`).
- add `data-test-subj` prop to panels for consumers writing tests.
- updates to copy, documentation, tests, storybook.

---------

Co-authored-by: florent-leborgne <[email protected]>
(cherry picked from commit c4fa63f)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
9.0

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 Mar 27, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…#216211)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[unification] Updates to AI Assistant Calls to Action
(#216105)](#216105)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Clint Andrew
Hall","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-03-27T17:11:57Z","message":"[unification]
Updates to AI Assistant Calls to Action (#216105)\n\n## Summary\n\nThis
PR addresses feedback and includes a number of changes:\n\n- allow a CTA
to be offset by the height of the `AssistantBeacon` when\ncentered
vertically, (e.g. in a tall `FlexItem`).\n- add `data-test-subj` prop to
panels for consumers writing tests.\n- updates to copy, documentation,
tests, storybook.\n\n---------\n\nCo-authored-by: florent-leborgne
<[email protected]>","sha":"c4fa63f1213e1a2105f0ab0a773be46cb0ce894c","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["review","release_note:skip","Team:SharedUX","backport:prev-minor","Feature:Security
Assistant","Team:Obs AI Assistant","v9.1.0"],"title":"[unification]
Updates to AI Assistant Calls to
Action","number":216105,"url":"https://github.com/elastic/kibana/pull/216105","mergeCommit":{"message":"[unification]
Updates to AI Assistant Calls to Action (#216105)\n\n## Summary\n\nThis
PR addresses feedback and includes a number of changes:\n\n- allow a CTA
to be offset by the height of the `AssistantBeacon` when\ncentered
vertically, (e.g. in a tall `FlexItem`).\n- add `data-test-subj` prop to
panels for consumers writing tests.\n- updates to copy, documentation,
tests, storybook.\n\n---------\n\nCo-authored-by: florent-leborgne
<[email protected]>","sha":"c4fa63f1213e1a2105f0ab0a773be46cb0ce894c"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/216105","number":216105,"mergeCommit":{"message":"[unification]
Updates to AI Assistant Calls to Action (#216105)\n\n## Summary\n\nThis
PR addresses feedback and includes a number of changes:\n\n- allow a CTA
to be offset by the height of the `AssistantBeacon` when\ncentered
vertically, (e.g. in a tall `FlexItem`).\n- add `data-test-subj` prop to
panels for consumers writing tests.\n- updates to copy, documentation,
tests, storybook.\n\n---------\n\nCo-authored-by: florent-leborgne
<[email protected]>","sha":"c4fa63f1213e1a2105f0ab0a773be46cb0ce894c"}}]}]
BACKPORT-->

Co-authored-by: Clint Andrew Hall <[email protected]>
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Mar 31, 2025
## Summary

This PR addresses feedback and includes a number of changes:

- allow a CTA to be offset by the height of the `AssistantBeacon` when
centered vertically, (e.g. in a tall `FlexItem`).
- add `data-test-subj` prop to panels for consumers writing tests.
- updates to copy, documentation, tests, storybook.

---------

Co-authored-by: florent-leborgne <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) Feature:Security Assistant Security Assistant release_note:skip Skip the PR/issue when compiling release notes review Team:Obs AI Assistant Observability AI Assistant Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v9.0.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants