feat: add deployment status handling#1342
Merged
nilsgstrabo merged 11 commits intomasterfrom Mar 30, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds deployment status awareness to the UI by extending the Radix API client types and wiring deployment status into environment and application status presentations.
Changes:
- Update
getSearchApplicationsto request environments and adjustApplicationSummaryto exposeenvironments(incl. deployment status) instead ofenvironmentActiveComponents. - Introduce a
DeploymentStatusBadgeand surface deployment status + failure reason in the environment overview and environment summary cards. - Refactor environment/app status aggregation utilities to incorporate deployment status alongside component/replica health.
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/store/radix-api.ts | Updates generated API typings and query args to include environments and deployment status; removes legacy search mutation/types. |
| src/components/status-badges/deployment-status-badge.tsx | Adds a new badge component for deployment reconciliation status. |
| src/components/page-environment/environment-overview.tsx | Shows deployment status badge and renders deployment failure reason when status is Failed. |
| src/components/page-deployment/dev.tsx | Updates dev fixture deployments to include required status. |
| src/components/page-deployment-job-component/dev.tsx | Updates dev fixture deployments to include required status. |
| src/components/page-deployment-component/dev.tsx | Updates dev fixture deployments to include required status. |
| src/components/job-overview/dev.tsx | Updates dev fixture deployment summaries to include required status. |
| src/components/environments-summary/environment-status-utils.ts | Adds deployment-status aggregation and renames/refactors component/replica aggregation helpers. |
| src/components/environments-summary/environment-headers.tsx | Adds deployment status into the environment status header aggregation. |
| src/components/environments-summary/environment-card.tsx | Passes deployment status through to the header component. |
| src/components/environments-summary/dev.tsx | Updates environment/dev fixtures to include deployment status. |
| src/components/deployments-list/dev.tsx | Updates deployment list fixtures to include status. |
| src/components/component/secrets/dev.tsx | Updates environment fixture activeDeployment to include status. |
| src/components/app-list/index.tsx | Switches search query params to includeEnvironments/includeLatestJobSummary booleans and passes environments through. |
| src/components/app-list-item/index.tsx | Reworks status aggregation to use environments (deployment + components + replicas). |
| proxy/server.dev.conf | Changes dev proxy default upstream to host.docker.internal. |
| package-lock.json | Lockfile updates (peer flags/metadata changes). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jacobsolbergholm
approved these changes
Mar 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.