Skip to content

DOCSP-41138: Distinct #24

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 4 commits into from
Aug 2, 2024
Merged

Conversation

mcmorisi
Copy link
Collaborator

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-41138
Staging - https://preview-mongodbmcmorisi.gatsbyjs.io/kotlin-sync/DOCSP-41138-distinct/read/distinct/

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?

Copy link

netlify bot commented Jul 30, 2024

👷 Deploy request for docs-kotlin-sync pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5f16ec1

Copy link
Collaborator

@jordan-smith721 jordan-smith721 left a comment

Choose a reason for hiding this comment

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

LGTM!

--------

Within a collection, different documents might contain different values for a single field.
For example, one ``restaurant`` document has a ``borough`` value of ``"Manhattan"``, and
Copy link
Collaborator

Choose a reason for hiding this comment

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

S:

Suggested change
For example, one ``restaurant`` document has a ``borough`` value of ``"Manhattan"``, and
For example, one document in the ``restaurant`` collection has a ``borough`` value of ``"Manhattan"``, and

free MongoDB Atlas cluster and load the sample datasets, see the
:atlas:`Get Started with Atlas </getting-started>` guide.

The following {+language+} data class models the document in this collection:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The following {+language+} data class models the document in this collection:
The following {+language+} data class models the documents in this collection:

@mcmorisi mcmorisi requested review from a team and vbabanin and removed request for a team July 30, 2024 17:22
Copy link
Member

@vbabanin vbabanin left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmorisi mcmorisi merged commit ae8646c into mongodb:master Aug 2, 2024
0 of 2 checks passed
@mcmorisi mcmorisi deleted the DOCSP-41138-distinct branch August 2, 2024 13:13
mcmorisi added a commit to mcmorisi/docs-kotlin-sync that referenced this pull request Aug 2, 2024
(cherry picked from commit ae8646c)
mcmorisi added a commit that referenced this pull request Aug 2, 2024
(cherry picked from commit ae8646c)
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.

3 participants