Skip to content

Remove unsupported operators by JSOnLogic#5394

Merged
emrysal merged 2 commits intomainfrom
fix-routing-forms-unsupported-operators
Nov 6, 2022
Merged

Remove unsupported operators by JSOnLogic#5394
emrysal merged 2 commits intomainfrom
fix-routing-forms-unsupported-operators

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

@hariombalhara hariombalhara commented Nov 6, 2022

What does this PR do?

Fixes Bug: Remove unsupported operators(startsWith and endsWith) by JSONLogic. react-query-awesome-builder supports them but not JSONLogic that we use for rule evaluation.
Found this bug while building reporting for all operators

  • These would be the operators that would remain. Contains is still supported and should be enough for now.

Screenshot 2022-11-06 at 11 40 06 AM

  • Rules that might be using it already (just in case) actually always evaluate to true, so removing these operators shouldn't have an effect on them.

Environment: Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • They should stop showing up in select as an operator

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 6, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Nov 6, 2022 at 6:37AM (UTC)

@what-the-diff
Copy link
Copy Markdown

what-the-diff Bot commented Nov 6, 2022

  • Removed the operators "starts_with" and "ends_with".

@hariombalhara hariombalhara marked this pull request as ready for review November 6, 2022 06:11
@hariombalhara hariombalhara added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Nov 6, 2022
@emrysal emrysal merged commit 7625950 into main Nov 6, 2022
@emrysal emrysal deleted the fix-routing-forms-unsupported-operators branch November 6, 2022 14:33
haffla pushed a commit to tourlane/cal.com that referenced this pull request Nov 22, 2022
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants