Skip to content

Fix typos in Field-Caps documentation #38580

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

Merged
merged 2 commits into from
Feb 11, 2019
Merged

Fix typos in Field-Caps documentation #38580

merged 2 commits into from
Feb 11, 2019

Conversation

dmeiss
Copy link
Contributor

@dmeiss dmeiss commented Feb 7, 2019

No description provided.

@colings86 colings86 added the :Search Foundations/Mapping Index mappings, including merging and defining field types label Feb 8, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

Copy link
Member

@cbuescher cbuescher left a comment

Choose a reason for hiding this comment

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

Hi @dmeiss, thanks for opening this PR to add changes to our docs. I left a small suggestion for improvement, would you be able to add those changes to the PR?


[source,js]
--------------------------------------------------
GET _field_caps?fields=rating
--------------------------------------------------
// CONSOLE

... but the request can also be restricted to specific indices:
...but the request can also be restricted to specific indices:
Copy link
Member

Choose a reason for hiding this comment

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

How about omitting the dots altogether, starting a new sentence with "The requestst can...". I think that would be much better.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree -- good suggestion. I'll make the change.

@cbuescher cbuescher self-assigned this Feb 11, 2019
@cbuescher cbuescher changed the title Fixed typos in text Fix typos in Field-Caps documentation Feb 11, 2019
@cbuescher cbuescher added >docs General docs changes v6.6.1 labels Feb 11, 2019
Reworded because asciidoc was formatting the ellipse/space as a numbered list.
Copy link
Member

@cbuescher cbuescher left a comment

Choose a reason for hiding this comment

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

Great, thanks @dmeiss for the additional changes. Will merge this after running some tests.

@cbuescher
Copy link
Member

@elasticmachine test this please

@cbuescher cbuescher merged commit 337a29c into elastic:6.6 Feb 11, 2019
cbuescher pushed a commit that referenced this pull request Feb 11, 2019
Fix typo in Field-Caps documentation

Reworded because asciidoc was formatting the ellipse/space as a numbered list.
cbuescher pushed a commit that referenced this pull request Feb 11, 2019
Fix typo in Field-Caps documentation

Reworded because asciidoc was formatting the ellipse/space as a numbered list.
cbuescher pushed a commit that referenced this pull request Feb 11, 2019
Fix typo in Field-Caps documentation

Reworded because asciidoc was formatting the ellipse/space as a numbered list.
cbuescher pushed a commit that referenced this pull request Feb 11, 2019
Fix typo in Field-Caps documentation

Reworded because asciidoc was formatting the ellipse/space as a numbered list.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :Search Foundations/Mapping Index mappings, including merging and defining field types v6.6.1 v6.7.0 v7.0.0-beta1 v7.2.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants