-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[unification] Updates to AI Assistant Calls to Action #216105
Conversation
Pinging @elastic/appex-sharedux (Team:SharedUX) |
Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant) |
1dd9b98
to
1e74632
Compare
There was a problem hiding this 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!
.../ai-assistant/ai-assistant-cta/install_knowledge_base/install_knowledge_base.translations.ts
Outdated
Show resolved
Hide resolved
…install_knowledge_base/install_knowledge_base.translations.ts Co-authored-by: florent-leborgne <[email protected]>
Starting backport for target branches: 9.0 |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Async chunks
Page load bundle
Unknown metric groupsAPI count
History
|
## 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)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…#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]>
## 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]>
Summary
This PR addresses feedback and includes a number of changes:
AssistantBeacon
when centered vertically, (e.g. in a tallFlexItem
).data-test-subj
prop to panels for consumers writing tests.