villebro commented on code in PR #23481: URL: https://github.com/apache/superset/pull/23481#discussion_r1149188547
########## 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: I agree with @john-bodley , I think this could be less verbose. Could something like this work? > Add filters clauses to control the filter's source query. 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: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org