fardin-developer opened a new pull request, #32665:
URL: https://github.com/apache/superset/pull/32665

   <!---
   Please write the PR title following the conventions at 
https://www.conventionalcommits.org/en/v1.0.0/
   Example:
   fix(dashboard): load charts correctly
   -->
   Fixes: #32621
   
   ### SUMMARY
   This PR addresses an issue where, with Time Comparison enabled in the Table 
Chart, the metric name ignores the defined label and instead uses the metric 
key. This results in an unfriendly display name in the UI.
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   <!--- Skip this if not applicable -->
   
   ###Changes
   - Introduced originalLabel in the DataColumnMeta interface to retain the 
correct label.
   - Updated processComparisonColumns to store originalLabel before modifying 
the metric label.
   - Ensured renderGroupingHeaders retrieves and displays the originalLabel 
correctly in grouped headers.
   
   ### BEFORE/AFTER SCREENSHOTS
   #### Before
   ![Screenshot 2025-03-14 at 11 33 05 
AM](https://github.com/user-attachments/assets/90d54d69-b31a-42a0-b9b5-fc06ce41ae93)
   
   #### After
   ![Screenshot 2025-03-14 at 12 29 16 
PM](https://github.com/user-attachments/assets/4d40269d-232e-4739-b7ed-6b229c420640)
   


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