Skip to content

change: [DPS-42071] - Improve errors handling in stream and destination form#13552

Open
kagora-akamai wants to merge 2 commits intolinode:developfrom
kagora-akamai:feature/DPS-42071-improve-errors-handling-in-stream-and-destination-form
Open

change: [DPS-42071] - Improve errors handling in stream and destination form#13552
kagora-akamai wants to merge 2 commits intolinode:developfrom
kagora-akamai:feature/DPS-42071-improve-errors-handling-in-stream-and-destination-form

Conversation

@kagora-akamai
Copy link
Copy Markdown
Contributor

Description 📝

On create/edit stream/destination show error provided in API response.

Changes 🔄

  • on Create/Edit Stream/Destination action show snackbar with error message from API response if provided.

Scope 🚢

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

  • Some customers (e.g. in Beta or Limited Availability)

Target release date 🗓️

April 2026

Preview 📷

Before After
Screenshot 2026-04-01 at 13 52 57 Screenshot 2026-04-01 at 13 44 13

How to test 🧪

Prerequisites

Launch manager with enabled MSW.
Update mock for createStream in mocks/presets/crud/handlers/delivery.ts to return error:

export const createStreams = (mockState: MockState) => [
  http.post(
    '*/v4/monitor/streams',
    async ({ request }): Promise<StrictResponse<APIErrorResponse | Stream>> => {
      return makeErrorResponse(
        'You\u2019ve reached the limit of streams you can create. If you have any questions, please contact support.'
      );
    }
  ),
];

Verification steps

  • If there is no Destinations created, navigate to logs delivery Destination Create page and create Destination.
  • Navigate to logs delivery Stream Create page and create stream with existing destination. See snackbar error with message provided in mock code above.
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


  • 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

@kagora-akamai kagora-akamai marked this pull request as ready for review April 1, 2026 13:09
@kagora-akamai kagora-akamai requested review from a team as code owners April 1, 2026 13:09
@kagora-akamai kagora-akamai requested review from bbanucha-akamai, dmcintyr-akamai, mduda-akamai and mikeZuber-akamai and removed request for a team April 1, 2026 13:09
@linode-gh-bot
Copy link
Copy Markdown
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing901 Passing11 Skipped39m 10s

Details

Failing Tests
SpecTest
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can update bucket access

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

3 participants