Skip to content

Specify top-level filtering #50

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

Closed
bdcodestuff opened this issue Oct 19, 2022 · 3 comments
Closed

Specify top-level filtering #50

bdcodestuff opened this issue Oct 19, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@bdcodestuff
Copy link

Hi -- the new changes in the latest release that enable resource embedding have been working great, thanks again!

I had one request though. It seems that by default, the postgrest-csharp library is adding top-level row filtering by inserting !inner in the query a la https://postgrest.org/en/stable/api.html#embedding-with-top-level-filtering. This ignores any top level rows for which there are no nested resources. It would be great to have more fine grained control over this. In my use case I would like all top level rows returned even if the nested resources contains no rows. Is there currently a way to control this, and if not could it be added?

@acupofjose
Copy link
Contributor

@bdcodestuff can you look at the PR? Would that do what you're expecting?

@bdcodestuff
Copy link
Author

Yes, that looks perfect! Thanks for the quick reply.

@acupofjose
Copy link
Contributor

@bdcodestuff happy to! I changed it to a boolean value instead of an enum (see pull). But it's available in version 2.1.1!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants