Skip to content

upcoming: [DPS-41443] - Metrics widget dimension filter and Group By Icons disabled tool tips#13496

Merged
santoshp210-akamai merged 13 commits intolinode:developfrom
santoshp210-akamai:feature/group_by_disabled_tooltip_text
Mar 24, 2026
Merged

upcoming: [DPS-41443] - Metrics widget dimension filter and Group By Icons disabled tool tips#13496
santoshp210-akamai merged 13 commits intolinode:developfrom
santoshp210-akamai:feature/group_by_disabled_tooltip_text

Conversation

@santoshp210-akamai
Copy link
Contributor

@santoshp210-akamai santoshp210-akamai commented Mar 13, 2026

Description 📝

Add disabled specific tool tip messages to DimensionFilters and Group By icons in ACLP-Metrics dashboards

Changes 🔄

  • When there are no dimension options, the Dimension Filter icon tooltip now reads "No dimensions available for filtering" instead of "Dimension Filters".
  • When there are no group-by dimensions (or they are loading), the Group By icon tooltip now reads "No dimensions available for grouping" instead of "Group By".
  • Tests are updated/added to cover both new disabled-state tooltip messages.
  • Additional test cases to cover the group-by icon in Widgets https://track.akamai.com/jira/browse/DPS-41508

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

28th March

Preview 📷

Before After
image image
image image

How to test 🧪

Prerequisites

(How to setup test environment)

  • In any Environment, Select Metrics under Monitor

Reproduction steps

(How to reproduce the issue, if applicable)

  • ...
  • ...

Verification steps

(How to verify changes)

  • Verify the updated tooltips texts are visible when the icons are disabled
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

Copy link
Collaborator

@kmuddapo kmuddapo left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Mar 16, 2026
@santoshp210-akamai santoshp210-akamai requested a review from a team as a code owner March 24, 2026 04:37
@santoshp210-akamai
Copy link
Contributor Author

The pipeline for the cloud-manager-tests is failing. Since the past failing tests are not failing due to the changes of this PR. I will be merging this.

@santoshp210-akamai santoshp210-akamai merged commit 0bff8eb into linode:develop Mar 24, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants