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

ExcludeKeys from nested objects in queries #7390

Closed
3 tasks done
pixael opened this issue May 19, 2021 · 6 comments
Closed
3 tasks done

ExcludeKeys from nested objects in queries #7390

pixael opened this issue May 19, 2021 · 6 comments
Labels
state:duplicate Duplicate of already reported issue

Comments

@pixael
Copy link

pixael commented May 19, 2021

New Feature / Enhancement Checklist

Current Limitation

Excluding keys in nested objects.

Feature / Enhancement Description

Exclude keys of nested objects using the syntax object.key.

Example Use Case

curl -X GET \
  -H "X-Parse-Application-Id: ${APPLICATION_ID}" \
  -H "X-Parse-REST-API-Key: ${REST_API_KEY}" \
  -G \
  --data-urlencode 'excludeKeys=player.profile' \
  https://YOUR.PARSE-SERVER.HERE/parse/classes/GameScore/Ed1nuqPvcm

Alternatives / Workarounds

I've not found any workaround. I'd need to esclude a heavy nested object from a query but can't at the moment.

@mtrezza
Copy link
Member

mtrezza commented May 19, 2021

Thanks for suggesting.

Excluding fields in embedded documents is natively supported by MongoDB and seems to be a useful addition to Parse Server. Feel free to open a PR to add this feature and ping if you need any help.

I flagged this as enhancement as it enhances the existing functionality of excluding fields to accept nested fields.

@mtrezza mtrezza added type:feature New feature or improvement of existing feature 🧬 enhancement and removed type:feature New feature or improvement of existing feature labels May 19, 2021
@cbaker6
Copy link
Contributor

cbaker6 commented May 20, 2021

This is already fixed in #7242

@pixael
Copy link
Author

pixael commented May 20, 2021

This is already fixed in #7242

Thanks Corey for making this change, I see that the PR is still in review... I hope it will be merged soon.

@mtrezza
Copy link
Member

mtrezza commented May 20, 2021

@cbaker6 Where is this described in #7242 or the related issue #7245?

According to the descriptions there they seem to focus on a different issue.

@cbaker6
Copy link
Contributor

cbaker6 commented May 20, 2021

@cbaker6 Where is this described in #7242 or the related issue #7245?

According to the descriptions there they seem to focus on a different issue.

It’s mentioned at the bottom of todos and linked to a related issues:

Fix excluding nested fields (Fix #7030)

In that issue, the last comment is from @dplewis asks me to look into it because of the PR I had open. You can see when I originally mentioned and the Diamond responds to my resolved comments on the issue shortly after.

The fix covers the issue mentioned here in the testcases, as well as deeper nested. This issue was also brought up again on the forum https://community.parseplatform.org/t/deselect-pointer-key/1592

@mtrezza
Copy link
Member

mtrezza commented May 20, 2021

Thanks, so I am closing this as duplicate of #7030.

@mtrezza mtrezza added state:duplicate Duplicate of already reported issue and removed 🧬 enhancement labels May 20, 2021
@mtrezza mtrezza closed this as completed May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:duplicate Duplicate of already reported issue
Projects
None yet
Development

No branches or pull requests

3 participants