Skip to content

[8.8] [Guided onboarding] Fix the missing help link (#156399)#156550

Merged
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-156399
May 3, 2023
Merged

[8.8] [Guided onboarding] Fix the missing help link (#156399)#156550
kibanamachine merged 1 commit intoelastic:8.8from
kibanamachine:backport/8.8/pr-156399

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.8:

Questions ?

Please refer to the Backport tool documentation

## Summary

This PR fixes a bug introduced in
elastic#155065 that I noticed when
testing guided onboarding for v8.9.0: the help link was missing from the
help menu. I added a functional test for the help link but we also need
a test that the link is hidden when the user doesn't have access to
guided onboarding (opened an issue for that
[here](elastic#156410)).

### How to test
1. Add xpack.cloud.id: 'testID' to /config/kibana.dev.yml
2. Start ES with yarn es snapshot and Kibana with `yarn start``
3. Check that the help link "Setup guides" is displayed in the help menu

### Screenshot
<img width="336" alt="Screenshot 2023-05-02 at 17 27 20"
src="https://user-images.githubusercontent.com/6585477/235712599-812b422d-3092-45b3-a726-37f6b90e81c1.png">

(cherry picked from commit d00e2a3)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 3, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) May 3, 2023 14:06
@kibanamachine kibanamachine merged commit a5921f0 into elastic:8.8 May 3, 2023
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #21 / visualize app visual builder Time Series "before each" hook for "should display correct data for the selected interval"

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 399 402 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
securitySolution 479 482 +3
total +5

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @yuliacech

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants