8000 fix: required dashboard filters for SQL charts by ZeRego · Pull Request #15234 · lightdash/lightdash · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: required dashboard filters for SQL charts #15234

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

Conversation

ZeRego
Copy link
Collaborator
@ZeRego ZeRego commented Jun 9, 2025

Closes: #15161

Description:

Before:
Screenshot 2025-06-09 at 11 51 25

After:
Screenshot 2025-06-09 at 12 48 10
Screenshot 2025-06-09 at 12 48 37

AI Description:

Added support for fallback type in dashboard filter targets to improve filter handling when fields or columns are not immediately available. This allows filters to function properly even when the underlying data structure is still loading or when using SQL columns.

Key changes:

  • Added fallbackType property to DashboardFieldTarget to infer filter types when fields/columns aren't available
  • Improved filter display logic to use fallback types when needed
  • Fixed filter configuration to properly handle cases where field data is still loading
  • Reordered some type definitions for better organization

Copy link
Collaborator Author
ZeRego commented Jun 9, 2025

@ZeRego ZeRego requested review from magnew and pimmey June 9, 2025 11:52
@ZeRego ZeRego changed the title feat: add fallbackType to DashboardFieldTarget for SQL column filters fix: required dashboard filters for SQL charts Jun 9, 2025
@ZeRego ZeRego linked an issue Jun 10, 2025 that may be closed by this pull request
Base automatically changed from refactor_split_Filter_component_into_Filter_and_AddFilterButton to main June 11, 2025 09:01
Copy link
Contributor
@magnew magnew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Tested locally -- looks good!

@ZeRego ZeRego force-pushed the feat_add_fallbackType_to_DashboardFieldTarget_for_SQL_column_filters branch from 6dc0d9d to 3e139cd Compare June 11, 2025 10:51
Copy link

Your preview environment pr-15234 has been deployed.

Preview environment endpoints are available at:

Copy link

@ZeRego ZeRego merged commit ad15d87 into main Jun 11, 2025
31 of 32 checks passed
@ZeRego ZeRego deleted the feat_add_fallbackType_to_DashboardFieldTarget_for_SQL_column_filters branch June 11, 2025 11:53
lightdash-bot pushed a commit that referenced this pull request Jun 11, 2025
# [0.1705.0](0.1704.0...0.1705.0) (2025-06-11)

### Features

* add fallbackType to DashboardFieldTarget for SQL column filters ([#15234](#15234)) ([ad15d87](ad15d87))
@lightdash-bot
Copy link
Collaborator

🎉 This PR is included in version 0.1705.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Required filters on a dashboard with just SQL charts can't be applied
3 participants
0