Skip to content

Conversation

@kagora-akamai
Copy link
Contributor

@kagora-akamai kagora-akamai commented Oct 6, 2025

Description 📝

Add info about stream provisioning time and change status labels

Changes 🔄

  • Stream create summary - add info about provisioning time after create
  • Add info about provisioning time to Success toast after Stream Created
  • Change labels for stream status from enabled/disabled to active/inactive

Scope 🚢

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

  • No customers / Not applicable

Target release date 🗓️

October 2025

Preview 📷

Before After
Screenshot 2025-10-06 at 15 16 06 Screenshot 2025-10-06 at 15 15 30
Screenshot 2025-10-06 at 15 21 46 Screenshot 2025-10-06 at 15 17 55 1
Screenshot 2025-10-06 at 15 20 57 Screenshot 2025-10-06 at 15 20 28

How to test 🧪

Prerequisites

  • Enable Mock CRUD in devtools

Verification steps

  • Navigate to /logs/delivery/streams/create - check info text above Test Connection button
  • Create stream and see updated toast message
  • Check the status in stream table to be Active/Inactive, updated action in menu action and status select options above the table
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
📱 Providing mobile 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

@kagora-akamai kagora-akamai self-assigned this Oct 6, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing837 Passing11 Skipped38m 37s

Details

Failing Tests
SpecTest
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts"

@kagora-akamai kagora-akamai marked this pull request as ready for review October 8, 2025 15:25
@kagora-akamai kagora-akamai requested a review from a team as a code owner October 8, 2025 15:25
@kagora-akamai kagora-akamai added the Add'tl Approval Needed Waiting on another approval! label Oct 9, 2025
@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Oct 13, 2025
@jaalah-akamai jaalah-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Oct 13, 2025
@jaalah-akamai
Copy link
Contributor

✅ Looks good, simple and straight forward changes

@mikeZuber-akamai mikeZuber-akamai merged commit 7e1e335 into linode:develop Oct 14, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Oct 14, 2025
corya-akamai pushed a commit to corya-akamai/manager that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! Logs

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants