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

docs(NODE-6483): Add documentation for countDocuments $match operator #4342

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

aditi-khare-mongoDB
Copy link
Contributor

Description

Add docs in from drivers ticket.

What is changing?

See above

Is there new documentation needed for these changes?

Only API docs

What is the motivation for this change?

DRIVERS-2728

Release Highlight

Document that countDocuments() uses $match and may not support the same filters as find/count commands

The API documentation for Collection.countDocuments now documents restrictions for filters in the countDocuments API that result from the API using aggregation pipeline.

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@durran durran changed the title docs(NODE-6485): Add documentation for countDocuments $match operator docs(NODE-6483): Add documentation for countDocuments $match operator Nov 27, 2024
@nbbeeken nbbeeken self-assigned this Dec 10, 2024
@nbbeeken nbbeeken added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Dec 10, 2024
@nbbeeken nbbeeken merged commit bc41f2e into main Dec 10, 2024
29 of 31 checks passed
@nbbeeken nbbeeken deleted the NODE-6483 branch December 10, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Primary Review In Review with primary reviewer, not yet ready for team's eyes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants