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

Adding initial User Behavior Insights documentation #6625

Closed
wants to merge 4 commits into from

Conversation

jzonthemtn
Copy link
Contributor

Description

Adds the initial documentation for the User Behavior Insights (UBI) plugin.

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

Thank you, @jzonthemtn! Left some suggestions.

_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
_search-plugins/user-behavior-insights/index.md Outdated Show resolved Hide resolved
@jzonthemtn
Copy link
Contributor Author

Thanks @kolchfa-aws! I appreciate the review and the suggestions.

Signed-off-by: jzonthemtn <[email protected]>
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you, @jzonthemtn! I'll move this to editorial review.

Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Jeff Zemerick <[email protected]>
@hdhalter
Copy link
Contributor

hdhalter commented Mar 7, 2024

Hi @kolchfa-aws - Let's hold off on editorial, as they may be adding more content before 2.13. Since this is their first doc PR, they wanted to get some early guidance to make sure they were doing it right. Thanks.

@hdhalter hdhalter added 2 - In progress Issue/PR: The issue or PR is in progress. and removed 4 - Doc review PR: Doc review in progress labels Mar 7, 2024
@hdhalter
Copy link
Contributor

hdhalter commented Jun 6, 2024

@jzonthemtn , @RasonJ , should we close this PR in favor of #7284?

@epugh
Copy link
Collaborator

epugh commented Jun 7, 2024

Closing this PR in favour of #7284

@epugh epugh closed this Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 - In progress Issue/PR: The issue or PR is in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants