Skip to content

Conversation

@ananya-agarwal
Copy link
Collaborator

@ananya-agarwal ananya-agarwal commented Oct 27, 2025

What changes were proposed in this pull request?

There were some final changes left that were to made in the Bulk Edit Column Modal as below:

  1. Fix inconsistent spacing between dropdown and table in MetricsTab
  2. Extract shared metric labels and show user-friendly names in dropdown
  3. Stick the left side tabs on the Quick Start page

How was this patch tested?

Manual testing. Please find the video of the changes made attached.

Screen.Recording.2025-10-27.at.3.36.31.PM.mov

Please review Hue Contributing Guide before opening a pull request.

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

✅ Test files were modified. Ensure that the tests cover all relevant changes. ✅

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

UI Code Coverage Report

Lines Statements Branches Functions
Coverage: 33%
39.89% (31308/78476) 31.56% (14610/46281) 25.89% (2355/9096)

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

Coverage

Backend Code Coverage Report •
FileStmtsMissCoverMissing
TOTAL569172893949% 
report-only-changed-files is enabled. No files were changed during this commit :)

@ananya-agarwal ananya-agarwal enabled auto-merge (squash) November 10, 2025 10:09
Copy link
Collaborator

@bjornalm bjornalm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

Copy link
Contributor

@JohanAhlen JohanAhlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ananya-agarwal ananya-agarwal merged commit 5835719 into master Nov 11, 2025
9 checks passed
@ananya-agarwal ananya-agarwal deleted the nitsAdmin branch November 11, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants