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

Add support for pagination on $everything #3944

Closed
healthbjk opened this issue Feb 13, 2024 · 1 comment · Fixed by #4319
Closed

Add support for pagination on $everything #3944

healthbjk opened this issue Feb 13, 2024 · 1 comment · Fixed by #4319

Comments

@healthbjk
Copy link

Why

Today the Medplum implementation of $everything returns all data in the Patient compartment
If the patient has many resources, this can be a large JSON response
Pagination is supported for other searches by Medplum

I think it would make sense to not paginate unless _count is provided? Optionally could implement a default pagination

What

Add support for _count to $everything in Medplum

@codyebberson
Copy link
Member

Thanks @healthbjk

Related discussion here: #4000

@codyebberson codyebberson added this to the April 30th, 2024 milestone Apr 2, 2024
medplumbot added a commit that referenced this issue Apr 4, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
codyebberson pushed a commit that referenced this issue Apr 4, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
github-merge-queue bot pushed a commit that referenced this issue Apr 5, 2024
Added sortStringArray utility function (#4297)Remove 'dist' from eligibility demo (#4300)
Fixes #3944 - patient everything pagination (#4296)
Fixed fragment with only one child warning (#4306)
Fixes #4299 - use correct cardinality on contentReference (#4304)
Add "New..." action in Medplum Provider app (#4303)
Updating case study material (#4305)
Drop unused columns from lookup tables (#4294)
Fixes #3905 - better graphql mutation error reporting (#4308)
Fixes #4309 - AWS Parameter Store support for required database SSL (#4310)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants