Fix:[DI-24064] - Fix navigation for metrics and alerts under Monitor

[ankita-akamai]

Description 📝

Hide or show 'Metrics' and 'Alerts' correctly based on all the relevant feature flags.

Changes 🔄

• Update primary navigation file.

Target release date 🗓️

• 25 March 2025

Preview 📷

here, aclp is enabled but none of the aclpalerting properties are true :

Before	After

How to test 🧪

Verification steps

(How to verify changes)

• Try modifying the hide prop in primary navigation file - PrimaryNav.tsx. Do hit and trial with the feature flags, making them true and false to identify all scenarios when these menu items will show/not show.
  Possible Scenarios:
  • if aclp feature flag is disabled, none of the menu items - 'metrics' or 'alerts' will be visible.
  • if aclp feature is enabled but aclpAlerting feature flag has none of its properties true, only 'metrics' menu item will be visible, 'alerts' will not be visible.
  • if aclp feature is enabled and aclpAlerting feature has any one or all of its properties true, both 'metrics' and 'alerts' menu items will be visible.

Make use of the mocks -

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

---

[nikhagra-akamai]

please add changeset

[github-actions]

Coverage Report: ✅
Base Coverage: 80.08%
Current Coverage: 80.08%

[pmakode-akamai]

It looks like the aclpAlerting feature flag is missing in DevTools. Could we add it so we can properly test it?

[ankita-akamai]

It looks like the aclpAlerting feature flag is missing in DevTools. Could we add it so we can properly test it?

@pmakode-akamai added the new feature flag - aclpAlerting in mocks , look for this label - CloudPulse Alerting

[pmakode-akamai]

thanks @ankita-akamai! Looks good 👍 I've just added a comment to simplify the logic.

[linode-gh-bot]

Cloud Manager UI test results

🎉 539 passing tests on test run #7 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
0 Failing	539 Passing	3 Skipped	112m 9s