Skip to content

change: [M3-9743] - Rename Analytics tab to Metrics tab on Linode details page - #12007

Merged
pmakode-akamai merged 3 commits into
linode:developfrom
pmakode-akamai:M3-9743-rename-analytics-tab-on-linode-details-page
Apr 11, 2025
Merged

change: [M3-9743] - Rename Analytics tab to Metrics tab on Linode details page#12007
pmakode-akamai merged 3 commits into
linode:developfrom
pmakode-akamai:M3-9743-rename-analytics-tab-on-linode-details-page

Conversation

@pmakode-akamai

@pmakode-akamai pmakode-akamai commented Apr 10, 2025

Copy link
Copy Markdown
Contributor

Description 📝

As part of the CloudPulse Linode Integration, the "Analytics" tab on the Linode detail page needs to be renamed to "Metrics".

Since this is a Global change, feature flag is not required.

Changes 🔄

  • Renamed Analytics tab to Metrics tab on Linode Details page
  • Updated the routeName for this tab
  • Random eslint fixes

Target release date 🗓️

N/A

Preview 📷

Before After
Screenshot 2025-04-10 at 5 42 27 PM Screenshot 2025-04-10 at 5 44 43 PM

How to test 🧪

  • All checks pass
  • Ensure the Analytics tab is now displayed as Metrics tab on Linode details page
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 unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@pmakode-akamai pmakode-akamai added the ACLP Integration CI (Cloud Interfaces) Support for CC (Core Compute) CloudPulse Integration label Apr 10, 2025
@pmakode-akamai pmakode-akamai self-assigned this Apr 10, 2025
@pmakode-akamai
pmakode-akamai force-pushed the M3-9743-rename-analytics-tab-on-linode-details-page branch from 34e92dc to 42489e8 Compare April 10, 2025 13:10
@pmakode-akamai
pmakode-akamai marked this pull request as ready for review April 10, 2025 13:13
@pmakode-akamai
pmakode-akamai requested a review from a team as a code owner April 10, 2025 13:13
@pmakode-akamai
pmakode-akamai requested review from harsh-akamai, hasyed-akamai and jaalah-akamai and removed request for a team April 10, 2025 13:13

@jaalah-akamai jaalah-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Looking good

@jaalah-akamai jaalah-akamai added the Add'tl Approval Needed Waiting on another approval! label Apr 10, 2025
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 540 passing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing540 Passing4 Skipped111m 16s

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Apr 11, 2025
@hasyed-akamai hasyed-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Apr 11, 2025
@pmakode-akamai
pmakode-akamai merged commit 80b5607 into linode:develop Apr 11, 2025
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ACLP Integration CI (Cloud Interfaces) Support for CC (Core Compute) CloudPulse Integration Approved Multiple approvals and ready to merge!

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants