Skip to content

Conversation

@rustagir
Copy link
Contributor

@rustagir rustagir commented Apr 10, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-48252

Staging Links

Autocomplete section in AS guide

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?
  • Are the facets and meta keywords accurate?
  • Are the page titles greater than 20 characters long and SEO relevant?

@netlify
Copy link

netlify bot commented Apr 10, 2025

Deploy Preview for docs-csharp ready!

Name Link
🔨 Latest commit ed0c07f
🔍 Latest deploy log https://app.netlify.com/sites/docs-csharp/deploys/67f91716b85ee50008fa2937
😎 Deploy Preview https://deploy-preview-586--docs-csharp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@stephmarie17 stephmarie17 left a comment

Choose a reason for hiding this comment

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

LGTM with a tiny suggestion ✅

:atlas:`How to Index Fields for Autocompletion </atlas-search/field-types/autocomplete-type/>`
in the Atlas documentation.

After you create the Search index, you must pass the index name to
Copy link
Collaborator

Choose a reason for hiding this comment

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

[s] I think if Search is capitalized here, it makes sense to include the full product name:

Suggested change
After you create the Search index, you must pass the index name to
After you create the Atlas Search index, you must pass the index name to

Otherwise it could be lower case:

Suggested change
After you create the Search index, you must pass the index name to
After you create the search index, you must pass the index name to

@rustagir rustagir merged commit 5eb8fc6 into mongodb:master Apr 11, 2025
6 checks passed
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
rustagir added a commit that referenced this pull request Apr 11, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix

(cherry picked from commit 5eb8fc6)
mongoKart pushed a commit to mongoKart/docs-csharp that referenced this pull request May 16, 2025
* DOCSP-48252: autocomplete index link/note

* SA small fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants