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

feat: _useEstimate parameter in count requests #261

Merged
merged 7 commits into from
Jan 30, 2024

Conversation

ThisIsDemetrio
Copy link
Collaborator

@ThisIsDemetrio ThisIsDemetrio commented Jan 30, 2024

Pull Request Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Description

Including querystring parameter _useEstimate in /count requests. When used, any other string parameter is ignored and the request of the total amount of documents is executed via estimatedDocumentCount of the MongoDB driver, effectively ignoring status and other filtering.

PR Checklist

  • The commit message follows our guidelines included in the CONTRIBUTING.md
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@ThisIsDemetrio ThisIsDemetrio marked this pull request as ready for review January 30, 2024 09:45
@ThisIsDemetrio ThisIsDemetrio changed the title Feat: _useEstimate parameter in count requests feat: _useEstimate parameter in count requests Jan 30, 2024
docs/10_Overview_and_Usage.md Outdated Show resolved Hide resolved
@ThisIsDemetrio ThisIsDemetrio merged commit 091144e into v6.x Jan 30, 2024
9 checks passed
@ThisIsDemetrio ThisIsDemetrio deleted the feat/estimated-document-count branch January 30, 2024 11:50
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