john-bodley commented on code in PR #23481:
URL: https://github.com/apache/superset/pull/23481#discussion_r1152402091
##########
superset-frontend/cypress-base/cypress/integration/dashboard/utils.ts:
##########
@@ -85,6 +85,7 @@ export const nativeFilterTooltips = {
multipleSelect: 'Allow selecting multiple values',
defaultValue:
'Default value must be set when "Filter value is required" is checked',
+ preFilter: `Limits the number of selectable values in a filter. This is
useful when you want to display a subset of the column's values either for
performance reasons or business rules. Some examples include limiting the
number of people displayed to a particular business unit or showing only people
registered in the last 90 days.`,
Review Comment:
@michael-s-molina and @villebro I _think_ this is more accurate:
> Add filter clauses to control the filter's source query, though only in
the context of the autocomplete i.e., these conditions do not impact how the
filter is applied to the dashboard. This is useful when you want to improve the
query's performance by only scanning a subset of the underlying data or limit
the available values displayed in the filter.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
