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

   ## **Sequence Diagram**
   
   The PR removes the explicit clearing of slice_id so matrixified cells 
inherit the parent's slice_id and dashboardId. This ensures each cell's API 
request includes the dashboard context so the backend permission check can 
verify chart membership and return data for embedded dashboards.
   
   ```mermaid
   sequenceDiagram
       participant Renderer
       participant MatrixifyGridGenerator
       participant Cell as "StatefulChart Cell"
       participant API
       participant Backend as "Backend (permission check)"
   
       Renderer->>MatrixifyGridGenerator: generateMatrixifyGrid(baseFormData 
with slice_id & dashboardId)
       MatrixifyGridGenerator->>Cell: create cell with formData (preserve 
slice_id & dashboardId)
       Cell->>API: Request chart data (formData includes slice_id + dashboardId)
       API->>Backend: Validate request (check dashboardId and slice_id for 
embedded user)
       Backend-->>API: Authorized -> chart data
       API-->>Cell: Return chart data
       Cell-->>Renderer: Render cell with data
   ```
   
   ---
   *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