codeant-ai-for-open-source[bot] commented on PR #38759:
URL: https://github.com/apache/superset/pull/38759#issuecomment-4097192093

   ## **Sequence Diagram**
   
   This PR adds a strict matrixify enable check before matrixify visibility, 
config extraction, and validation run. When old form data is loaded without 
matrixify enabled, matrixify logic is skipped so charts render without false 
validation failures.
   
   ```mermaid
   sequenceDiagram
       participant User
       participant Explore UI
       participant Matrixify Helpers
       participant Chart Renderer
   
       User->>Explore UI: Open chart with old form data
       Explore UI->>Matrixify Helpers: Evaluate matrixify visibility config and 
validation
       Matrixify Helpers->>Matrixify Helpers: Require matrixify enable true
       Matrixify Helpers-->>Explore UI: Return hidden state null config and no 
errors
       Explore UI->>Chart Renderer: Render chart without matrixify rules
       Chart Renderer-->>User: Chart loads successfully
   ```
   
   ---
   *Generated by [CodeAnt AI](https://codeant.ai)*
   


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