Skip to content

fix(eap-resolver): escape sequences should also be translated in a in query#104188

Merged
DominikB2014 merged 5 commits intomasterfrom
dominikbuszowiecki/browse-164-truncated-queries-affect-filtering-on-query-summary
Dec 2, 2025
Merged

fix(eap-resolver): escape sequences should also be translated in a in query#104188
DominikB2014 merged 5 commits intomasterfrom
dominikbuszowiecki/browse-164-truncated-queries-affect-filtering-on-query-summary

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 commented Dec 1, 2025

Resolves BROWSE-164

If you use a in query with escaped characters, we should also translate them for the elements in the array, the same as we do for a regular is query. This is so we never have discrepancies between the results of queries attribute:[a] and attribute:a

@DominikB2014 DominikB2014 requested a review from a team as a code owner December 1, 2025 20:11
@linear
Copy link
Copy Markdown

linear bot commented Dec 1, 2025

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 1, 2025
@DominikB2014 DominikB2014 changed the title fix(eap-resolver): escape sequences should also be translated in lists fix(eap-resolver): escape sequences should also be translated in a in query Dec 1, 2025
Copy link
Copy Markdown
Member

@wmak wmak left a comment

Choose a reason for hiding this comment

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

Can you add a test for arrays with numbers too? otherwise lgtm

@DominikB2014 DominikB2014 merged commit 01f12c9 into master Dec 2, 2025
66 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/browse-164-truncated-queries-affect-filtering-on-query-summary branch December 2, 2025 14:36
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants