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

Correct an example in the top-level suggester documentation. #30224

Merged
merged 1 commit into from
May 1, 2018

Conversation

jtibshirani
Copy link
Contributor

This seems innocuous, but can be quite confusing if you are new to suggesters.

@jtibshirani jtibshirani added >docs General docs changes v7.0.0 :Search Relevance/Suggesters "Did you mean" and suggestions as you type labels Apr 27, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@debadair
Copy link
Contributor

debadair commented May 1, 2018

Indeed! LGTM

@jtibshirani jtibshirani merged commit 5c9f084 into elastic:master May 1, 2018
@jtibshirani jtibshirani deleted the docs/suggester branch May 1, 2018 22:16
dnhatn added a commit that referenced this pull request May 2, 2018
* master: (68 commits)
  [DOCS] Removes X-Pack Elasticsearch release notes (#30272)
  Correct an example in the top-level suggester documentation. (#30224)
  [DOCS] Removes broken link
  [DOCS] Adds file realm configuration details (#30221)
  [DOCS] Adds PKI realm configuration details (#30225)
  Fix a reference to match_phrase_prefix in the match query docs. (#30282)
  Fix failure for validate API on a terms query (#29483)
  [DOCS] Fix 6.4-specific link in changelog (#30314)
  Remove RepositoriesMetaData variadic constructor (#29569)
  Test: increase authentication logging for debugging
  [DOCS] Removes redundant SAML realm settings (#30196)
  REST Client: Add Request object flavored methods (#29623)
  [DOCS] Adds changelog to Elasticsearch Reference (#30271)
  [DOCS] Fixes section error
  SQL: Teach the CLI to ignore empty commands (#30265)
  [DOCS] Adds Active Directory realm configuration details (#30223)
  [DOCS] Removes redundant file realm settings (#30192)
  [DOCS] Fixes users command name (#30275)
  Build: Move gradle wrapper jar to a dot dir (#30146)
  Build: Log a warning if disabling reindex-from-old (#30304)
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :Search Relevance/Suggesters "Did you mean" and suggestions as you type v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants