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.

Sign up for GitHub

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

Jump to bottom

Update heading on telemetry management section. #115425

Merged
merged 1 commit into from
Oct 19, 2021
Merged

Update heading on telemetry management section. #115425

merged 1 commit into from
Oct 19, 2021

Conversation

pjhampton
Copy link
Contributor

Summary

In the backref'd issue we deemed it appropriate to change the word 'statistics' with 'data' given the data solutions are now collecting to improve the user experience.

Screenshot 2021-10-18 at 18 40 54

### Checklist

Delete any items that are not applicable to this PR.

For maintainers

@pjhampton pjhampton added Feature:Telemetry v8.0.0 release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v7.16.0 labels Oct 18, 2021
@pjhampton pjhampton requested a review from a team as a code owner October 18, 2021 17:43
@kibanamachine
Copy link
Contributor

💚 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
telemetryManagementSection 10.3KB 10.3KB -24.0B

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

TinaHeiligers
TinaHeiligers approved these changes Oct 18, 2021
View reviewed changes
Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

LGTM!

Bamieh
Bamieh approved these changes Oct 19, 2021
View reviewed changes
Copy link
Member

@Bamieh Bamieh left a comment

Choose a reason for hiding this comment

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

LGTM

@pjhampton pjhampton merged commit fd4b85b into master Oct 19, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 19, 2021
@pjhampton @kibanamachine
Update heading on telemetry management section. (elastic#115425)
91114d0
@kibanamachine kibanamachine mentioned this pull request Oct 19, 2021
Merged
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
✅ 7.x

This backport PR will be merged automatically after passing CI.

@pjhampton pjhampton deleted the pjhampton/update-heading branch October 19, 2021 07:37
kibanamachine added a commit that referenced this pull request Oct 19, 2021
@kibanamachine @pjhampton
Update heading on telemetry management section. (#115425) (#115508)
8e6ba27 
Co-authored-by: Pete Hampton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TinaHeiligers TinaHeiligers TinaHeiligers approved these changes

@Bamieh Bamieh Bamieh approved these changes

Assignees
No one assigned
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes v7.16.0 v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pjhampton @kibanamachine @Bamieh @TinaHeiligers