vhogberg opened a new issue, #43086:
URL: https://github.com/apache/superset/issues/43086

   ### Bug description
   
   **The issue**
   The distribute across field uses `validateNonEmpty`, but its `initialValue` 
returns
   `[getTemporalColumns(state.datasource).defaultTemporalColumn]`. With no 
default temporal
   column that's `[undefined]`, an array of length 1, which passes validation 
and leaves you with a not so user-friendly error message.
   
   **Steps to reproduce**
   1. Create a Box Plot on a dataset with no default datetime column.
   2. Only `Metrics` is marked required; `Distribute across` is not.
   3. Fill in `Metrics` only and create chart -> error.
   4. Add then remove a column in `Distribute across` -> only now is it flagged 
as required.
   
   `Distribute across` should be be flagged as required on first load when 
empty. Datasets that do have a default datetime column auto-populate the 
control, so the issue isn't visible there.
   
   ### Superset version
   6.1
   
   ### Screenshots/recordings
   
   <img width="633" height="141" alt="Image" 
src="https://github.com/user-attachments/assets/09a938dd-de66-4028-9d26-c0bd2ec71f02";
 />
   
   <img width="345" height="427" alt="Image" 
src="https://github.com/user-attachments/assets/919cc633-569b-4944-a183-146de90c3dd2";
 />
   
   ### Superset version
   
   6.1.0
   
   ### Python version
   
   Not applicable
   
   ### Node version
   
   Not applicable
   
   ### Browser
   
   Chrome
   
   ### Additional context
   
   _No response_
   
   ### Checklist
   
   - [x] I have searched Superset docs and Slack and didn't find a solution to 
my problem.
   - [x] I have searched the GitHub issue tracker and didn't find a similar bug 
report.
   - [x] I have checked Superset's logs for errors and if I found a relevant 
Python stacktrace, I included it here as text in the "additional context" 
section.


-- 
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]

Reply via email to