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

[Guided onboarding] Updated the examples #141265

Merged

Conversation

yuliacech
Copy link
Contributor

Summary

Following a conversation with @stephmilovic, I updated the examples to only use the Observable for the active step state.

@elasticmachine
Copy link
Contributor

Pinging @elastic/platform-onboarding (Team:Journey/Onboarding)

@yuliacech yuliacech enabled auto-merge (squash) September 21, 2022 16:03
Copy link
Contributor

@alisonelizabeth alisonelizabeth left a comment

Choose a reason for hiding this comment

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

Thanks for updating this! Left one comment around changing the step ID due to the updated config changes made in #141233.


const [isTourStepOpen, setIsTourStepOpen] = useState<boolean>(false);

useEffect(() => {
const subscription = guidedOnboardingApi
?.isGuideStepActive$('search', 'search_experience')
Copy link
Contributor

Choose a reason for hiding this comment

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

I this this may need to change to browse_docs based on the changes I made in https://github.com/elastic/kibana/pull/141233/files

@yuliacech yuliacech added v8.6.0 and removed v8.5.0 labels Sep 26, 2022
Copy link
Contributor

@alisonelizabeth alisonelizabeth left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Page load bundle

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

id before after diff
guidedOnboarding 14.1KB 14.0KB -14.0B

History

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

@yuliacech yuliacech merged commit 85c8d37 into elastic:main Sep 26, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Sep 26, 2022
jennypavlova pushed a commit to jennypavlova/kibana that referenced this pull request Sep 27, 2022
@yuliacech yuliacech deleted the guided_onboarding/8.6_api_examples_fix branch November 22, 2022 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:fix Team:Journey/Onboarding Platform Journey Onboarding team v8.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants