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

Added support for AWS Document DB #208

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

slayer1551
Copy link

Since AWS Document DB does not support $facet and throws an error, I created a workaround for generating the pagination and results.

If the original call throws an error it tries the AWS workaround. Tested and it returns back filtered results with correct pagination

$facet is not available in AWS Document DB and throws an error.
Created an error check that if it fails with the standard query to try the AWS query to create the pages and filters.
Fixed the total page count
removed line uses for testing
@slayer1551 slayer1551 changed the title Added support for AWD Document DB Added support for AWS Document DB Jun 1, 2022
@devclickuk
Copy link

This will be amazing if this can get pushed, been wanting use documentDB with agendash for a while now

@DAmisonSSG
Copy link

Any update on this getting pushed?

@koresar
Copy link
Contributor

koresar commented Apr 11, 2023

Loving your workaround!
But could we please rewrite it to avoid double querying?
(Also, there is a merge conflict now. sorry!)

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.

None yet

4 participants