Skip to content

Conversation

@hiagolcm
Copy link
Contributor

What problem is this solving?

Previously, a character validation for special characters like &[]/?= was added to the productSearch query to support legacy behavior before intelligent search was introduced.

This same validation was mistakenly copied to the searchMetadata query. I've removed it, as testing confirmed that all special characters are now handled correctly and the validation is no longer needed.

How should this be manually tested?

Workspace

@hiagolcm hiagolcm requested a review from a team as a code owner August 22, 2025 19:38
@vtex-io-ci-cd
Copy link

vtex-io-ci-cd bot commented Aug 22, 2025

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@vtex-io-docs-bot
Copy link

vtex-io-docs-bot bot commented Aug 22, 2025

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

  • There's nothing new to document 🤔
  • I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

  • I just updated 🎉🎉

ArthurFerrao
ArthurFerrao previously approved these changes Sep 19, 2025
@hiagolcm hiagolcm force-pushed the bugfix/remove-invalid-character-validation-from-searchmetadata branch from 010516b to 13343df Compare October 29, 2025 18:23
@hiagolcm hiagolcm force-pushed the bugfix/remove-invalid-character-validation-from-searchmetadata branch from 590e4b2 to 366838a Compare October 29, 2025 18:36
@hiagolcm hiagolcm merged commit b0b3511 into main Oct 29, 2025
4 checks passed
@hiagolcm hiagolcm deleted the bugfix/remove-invalid-character-validation-from-searchmetadata branch October 29, 2025 19:36
@vtex-io-ci-cd
Copy link

vtex-io-ci-cd bot commented Oct 29, 2025

Your PR has been merged! App is being published. 🚀
Version 1.88.1 → 1.88.2

After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:

vtex deploy [email protected]

After that your app will be updated on all accounts.

For more information on the deployment process check the docs. 📖

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