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

   ## **Sequence Diagram**
   
   This PR fixes timeseries table sorting by exposing each numeric cell value 
through a data value wrapper and simplifying the numeric comparator. React 
table now receives valid values and correctly applies ascending or descending 
direction on its own.
   
   ```mermaid
   sequenceDiagram
       participant User
       participant TimeTable
       participant ValueCell
       participant ReactTable
       participant SortUtils
   
       User->>TimeTable: Render timeseries table
       TimeTable->>ValueCell: Compute cell value for each metric
       ValueCell-->>TimeTable: Render cell wrapped with data value
   
       User->>ReactTable: Click column header to sort
       ReactTable->>SortUtils: Compare row data values
       SortUtils-->>ReactTable: Return raw numeric comparison
       ReactTable->>ReactTable: Apply sort direction internally
       ReactTable-->>User: Show correctly reordered rows
   ```
   
   ---
   *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