-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
fix(Supabase Node): Allow for filtering on the same field multiple times #12429
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested Manually as well and it works perfectly 🎉
|
n8n Run #8576
Run Properties:
|
Project |
n8n
|
Branch Review |
master
|
Run status |
Passed #8576
|
Run duration | 04m 55s |
Commit |
d7cc789d79: 🌳 master 🖥️ browsers:node18.12.0-chrome107 🤖 PR User 🗃️ e2e/*
|
Committer | Dana |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
4
|
Pending |
0
|
Skipped |
0
|
Passing |
484
|
View all changes introduced in this branch ↗︎ |
|
1 similar comment
|
c1458c4
to
6f6d8a6
Compare
✅ All Cypress E2E specs passed |
Got released with |
1 similar comment
Got released with |
Got released with |
1 similar comment
Got released with |
Summary
This PR allows filtering on the same field multiple times
Workflow for testing:
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/NODE-2107/community-issue-manual-filtering-bug-with-supabase
Closes #11998
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)