Skip to content
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

Enhance Frontend validation and UX when adding or editing Charts #34

Open
nicolaes opened this issue Nov 25, 2023 · 1 comment
Open
Labels
enhancement 🚀 New feature or request

Comments

@nicolaes
Copy link

Is your feature request related to a problem? Please describe.
Configuring the columns for a new chart can end up in long trial and error sequences if you're not familiar with the data, the data types and the chart types.

Describe the solution you'd like

  1. X data column is required
  2. Replace the empty option in Y data column with "Row count"
    Currently, leaving an empty value defaults to COUNT(*), which is the same regardless of selected Y column
  3. Disable Y function dropdown if "Row count" is selected in Y data
  4. Remove "Count" from Y function dropdown
  5. Remove "Temporal data" field if it's not used

Describe alternatives you've considered
I did not think about other implementations.

Additional context
Screenshot 2023-11-25 at 16 30 22

@nicolaes nicolaes added the enhancement 🚀 New feature or request label Nov 25, 2023
@nicolaes
Copy link
Author

nicolaes commented Nov 25, 2023

Implementing 1st requirement fixes server errors when viewing a broken chart.

broken_chart

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🚀 New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant