Skip to content

Support query param operations on nested resources #634

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 7 commits into from
Nov 25, 2019

Conversation

maurei
Copy link
Member

@maurei maurei commented Nov 22, 2019

This closed #628 and partially addresses #627

Please see #627 for a the relevant discussion about this

  • Support includes on nested route
    - [ ] Support filter on nested route
    - [ ] Support sparse field selection on nested route
    - [ ] Support paging on nested route
  • Throw HTTP 400 when unsupported query params are used in nested resource routes

@maurei maurei force-pushed the fix/nested-resources branch from 50491ba to daab069 Compare November 22, 2019 11:47
@maurei maurei merged commit b5207ec into master Nov 25, 2019
@bart-degreed bart-degreed deleted the fix/nested-resources branch August 21, 2020 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Fetch include in deep URL passes query validation, but is not applied
2 participants