Skip to content

test[DI-28502]:- Notification channel Management - Listing Page#13204

Merged
santoshp210-akamai merged 8 commits intolinode:developfrom
agorthi-akamai:alert-notification-channels-listing_linode_December_15
Dec 18, 2025
Merged

test[DI-28502]:- Notification channel Management - Listing Page#13204
santoshp210-akamai merged 8 commits intolinode:developfrom
agorthi-akamai:alert-notification-channels-listing_linode_December_15

Conversation

@agorthi-akamai
Copy link
Contributor

@agorthi-akamai agorthi-akamai commented Dec 16, 2025

Description 📝

This PR implements the notification channels listing feature for CloudPulse alerts, enabling users to view and search through their configured notification channels with detailed information about each channel.

Changes 🔄

Added a complete notification channels listing page with table view, search functionality, and pagination
Implemented data fetching, filtering, and display logic for notification channels
Added comprehensive test coverage for the new components and features

| 📷 | 📷 |

How to test 🧪

pnpm cy:run -s "cypress/e2e/core/cloudpulse/alert-notification-channel-list.spec.ts"

Preview;

alert-notification-channel-list.spec.ts.mp4

♿ Providing accessibility suppor

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

@agorthi-akamai agorthi-akamai force-pushed the alert-notification-channels-listing_linode_December_15 branch from c4f348d to e438b6f Compare December 16, 2025 03:06
@agorthi-akamai agorthi-akamai marked this pull request as ready for review December 16, 2025 03:07
@agorthi-akamai agorthi-akamai requested a review from a team as a code owner December 16, 2025 03:07
Copy link
Contributor

@santoshp210-akamai santoshp210-akamai 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 Dec 16, 2025
Copy link
Contributor

@venkymano-akamai venkymano-akamai left a comment

Choose a reason for hiding this comment

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

LGTM, but lets see for sorting if we can achieve something like suggested

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 867 passing tests on test run #9 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing867 Passing11 Skipped48m 12s

@santoshp210-akamai santoshp210-akamai merged commit 807c7c1 into linode:develop Dec 18, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants