Skip to content

Conversation

@bnGil
Copy link
Contributor

@bnGil bnGil commented Oct 15, 2025

No description provided.

@bnGil bnGil force-pushed the gil/implement-filter branch from 9446a21 to 75eedac Compare October 27, 2025 09:32
case '$lte':
wixFilter = wixFilter.le(fieldPath, operandValue as any);
break;
case '$in':
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why $in case uses hasSome(), we already have case for $hasSome. and if we don't have in operator on filter, then we should not use it, the $in case is for other verticals who supports $in operator (e.g. stores: https://dev.wix.com/docs/sdk/backend-modules/stores/catalog-v3/products-v3/products-query-builder/in)

@bnGil bnGil merged commit 79cc5ce into main Oct 27, 2025
1 check passed
@bnGil bnGil deleted the gil/implement-filter branch October 27, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants