-
Notifications
You must be signed in to change notification settings - Fork 0
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
[BUG] Non published references are returned by DataSync when using include([Array of refs]) function and the UIDs are mixed up then. #50
Comments
@harshithad0703 As discussed I opened this as a new issue with the correct description of the error. |
Hi @idmedia-kaiser , For example:- FYI |
Hi @aman19K, as I wrote also in the ticket.If you check the demo And the issue is not the its not working, the issue is with unpublished entries getting returned partially and mixing up with existing entries
same resposne without the include(["fields"]
expected is this
|
Yes @idmedia-kaiser , We are checking the demo. As soon as we can reproduce the problem, we'll let you know. |
Hi @idmedia-kaiser ,We can reproduce the issue on my end, and it is functioning correctly now with this fix. Would you please try that and let me know if it functions properly on your end as well? |
Hi @aman19K, I just applied this fix locally, and it's not really working, its just removes the next to last element.
After the fix
|
Hi @idmedia-kaiser , We’re re-verifying the fix and will let you know if we able to reproduce it or need any help from your side. |
Hi @idmedia-kaiser cc: @aman19K @netrajpatel |
Hi @harshithad0703 @aman19K @netrajpatel , that looks good! blt983d6cdb2ec347cd is no longer present and bltf45bb3e8d8c04dd1 is there once and the correct field. After this fix
Please let us know when this will be released :) |
Thank you for confirming! It’s great to hear that the issue has been fixed and the correct field is now present This fix is scheduled to be released by 20th January. |
Hi, we are facing an issue with DataSync when using the include() function to include only specific references.
https://contentstack.github.io/datasync-filesystem-sdk/global.html#include
We have a form content type with a reference field for fields
https://app.contentstack.com/#!/stack/blt81b4b787a1850697/content-type/form/de-de/entry/blt0e8af3662cf665b0/edit?branch=main
Some of those references are not published. If we check in the DEV Tool widget we correctly don't get this fields back that are not published.
https://cdn.contentstack.io/v3/content_types/form/entries/blt0e8af3662cf665b0?api_key=blt81b4b787a1850697&access_token=&environment=development&locale=de-de
If we use the
include(["fields"])
function then the not published entries get mixed up with entries that are actually published and we get duplicated contentComposed query
This works correctly if we use replace
blogQuery = blogQuery.include(["fields"]);
with
blogQuery = blogQuery.includeReferences();
but we don't want all references but only some of them.
The text was updated successfully, but these errors were encountered: