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

[8.16] Updates phrasing when referring to pages (backport #2864) #2867

Merged
merged 3 commits into from
Nov 25, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Nov 6, 2024

Overview

This PR updates app and page references to align with the new style guide guidelines, ensuring consistency across both the classic and solution views to help users locate pages regardless of their chosen view. Additionally, the update removes interactive checkboxes, replacing them with bullet points.

Related issue

https://github.com/elastic/search-docs-team/issues/205


This is an automatic backport of pull request #2864 done by Mergify.

* Updates phrasing when referring to pages

* Fixes typos

* Replaces 'global search bar' with 'global search field'

(cherry picked from commit d015f34)

# Conflicts:
#	docs/en/stack/ml/anomaly-detection/ml-detect-categories.asciidoc
#	docs/en/stack/ml/anomaly-detection/ml-population-analysis.asciidoc
@mergify mergify bot requested a review from a team as a code owner November 6, 2024 11:44
Copy link
Contributor Author

mergify bot commented Nov 6, 2024

Cherry-pick of d015f34 has failed:

On branch mergify/bp/8.16/pr-2864
Your branch is up to date with 'origin/8.16'.

You are currently cherry-picking commit d015f34e.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/en/stack/ml/anomaly-detection/ml-ad-run-jobs.asciidoc
	modified:   docs/en/stack/ml/anomaly-detection/ml-jobs-from-visuals.asciidoc
	modified:   docs/en/stack/ml/anomaly-detection/ml-revert-model-snapshot.asciidoc
	modified:   docs/en/stack/ml/df-analytics/ml-dfa-shared.asciidoc
	modified:   docs/en/stack/ml/get-started/ml-gs-results.asciidoc
	modified:   docs/en/stack/ml/get-started/ml-gs-visualizer.asciidoc
	modified:   docs/en/stack/ml/nlp/ml-nlp-e5.asciidoc
	modified:   docs/en/stack/ml/nlp/ml-nlp-elser.asciidoc
	modified:   docs/en/stack/ml/nlp/ml-nlp-inference.asciidoc
	modified:   docs/en/stack/ml/nlp/ml-nlp-ner-example.asciidoc
	modified:   docs/en/stack/ml/setup.asciidoc

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	both modified:   docs/en/stack/ml/anomaly-detection/ml-detect-categories.asciidoc
	deleted by us:   docs/en/stack/ml/anomaly-detection/ml-population-analysis.asciidoc

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Copy link

github-actions bot commented Nov 6, 2024

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

Copy link
Contributor Author

mergify bot commented Nov 11, 2024

This pull request has not been merged yet. Could you please review and merge it @kosabogi? 🙏

1 similar comment
Copy link
Contributor Author

mergify bot commented Nov 18, 2024

This pull request has not been merged yet. Could you please review and merge it @kosabogi? 🙏

Copy link
Contributor Author

mergify bot commented Nov 25, 2024

This pull request has not been merged yet. Could you please review and merge it @kosabogi? 🙏

@kosabogi kosabogi merged commit e72d950 into 8.16 Nov 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant