-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 query binding drawer bindable options #4128
Conversation
…g sure correct ones are available.
Was just looking at this file - for some reason it has props for both |
packages/builder/src/components/integration/QueryBindingBuilder.svelte
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #4128 +/- ##
=======================================
Coverage 68.92% 68.92%
=======================================
Files 139 139
Lines 4634 4634
Branches 685 685
=======================================
Hits 3194 3194
Misses 1018 1018
Partials 422 422 Continue to review full report at Codecov.
|
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.
LGTM! Nice getting that fix up so quickly.
… variable with queryBindings.
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.
LGTM! Renaming makes things much more consistent.
Description
Fix for #4126 - wrong bindings were being passed to the drawer, making sure correct ones are available.