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
[explore] Include "Autocomplete Query Predicate" field to Table Editor #6171
[explore] Include "Autocomplete Query Predicate" field to Table Editor #6171
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6171 +/- ##
==========================================
+ Coverage 76.91% 76.91% +<.01%
==========================================
Files 47 47
Lines 9362 9363 +1
==========================================
+ Hits 7201 7202 +1
Misses 2161 2161
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. Just a small unit test suggestion.
@@ -336,6 +336,18 @@ export class DatasourceEditor extends React.PureComponent { | |||
descr={t('Whether to populate autocomplete filters options')} | |||
control={<CheckboxControl />} | |||
/> | |||
{this.state.isSqla && |
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.
Perhaps a quick unit test for the conditional rendering? :)
f39be7c
to
e03b445
Compare
Addressed comments, merging |
apache#6171) * Improvements around filters * Adding simple test (cherry picked from commit 5403f31)
apache#6171) * Improvements around filters * Adding simple test (cherry picked from commit 5403f31) (cherry picked from commit 2e3e063)
apache#6171) * Improvements around filters * Adding simple test (cherry picked from commit 5403f31) (cherry picked from commit 2e3e063)
apache#6171) * Improvements around filters * Adding simple test
@betodealmeida