Skip to content

feat(deployment-history): add service deployment and update environment deployment view#2422

Merged
RemiBonnet merged 6 commits intonew-navigationfrom
feat/new-nav/service-deployment-history
Feb 26, 2026
Merged

feat(deployment-history): add service deployment and update environment deployment view#2422
RemiBonnet merged 6 commits intonew-navigationfrom
feat/new-nav/service-deployment-history

Conversation

@RemiBonnet
Copy link
Member

@RemiBonnet RemiBonnet commented Feb 25, 2026

Summary

  • Add Service Deployments History view
  • Update Environment Deployments History view with new UI
  • Update DeploymentAction component and using Icon component instead of custom SVG

Screenshots / Recordings

Organization - Overview
Organization - Overview · 8 43am · 02-26

Testing

  • Changes tested locally in the relevant Console's pages and Storybooks
  • yarn test or yarn test -u (if you need to regenerate snapshots)
  • yarn format
  • yarn lint

PR Checklist

  • I followed naming, styling, and TypeScript rules (see .cursor/rules)
  • I performed a self-review (diff inspected, dead code removed)
  • I titled the PR using Conventional Commits with a scope when possible (e.g. feat(service): add new Terraform service) - required for semantic-release
  • I only kept necessary comments, written in English (watch for useless AI comments)
  • I involved a designer to validate UI changes if I am not a designer
  • I covered new business logic with tests (unit)
  • I confirmed CI is green (Codecov red can be accepted)
  • I reviewed and executed locally any AI-assisted code

…nt view

- Renamed 'Deployment' tab to 'Deployments' for consistency in navigation.
- Added a new route for service deployments, improving the structure of deployment-related views.
- Introduced a new `ServiceDeploymentList` component with enhanced loading states and UI consistency.
- Updated `EnvironmentDeploymentList` to include an empty state for better user feedback when no deployments are available.
- Refactored deployment-related tests to ensure accurate rendering and functionality.
- Removed deprecated deployment feature pages to streamline the codebase.
- Updated routeTree.gen.ts to include new routes for improved organization and clarity.
- Consolidated existing imports and added several new routes related to authentication, organization settings, and project management.
- Removed unused route references to streamline the codebase and improve maintainability.
@RemiBonnet
Copy link
Member Author

Qovery Preview

Qovery can create a Preview Environment for this PR.
To trigger its creation, please post a comment with one of the following command.

Command Blueprint environment
/qovery preview 15d69f24-9bc1-4a8d-80fe-d1bb1b2bcd00 New Navigation
/qovery preview {all|UUID1,UUID2,...} To preview multiple environments

This comment has been generated from Qovery AI 🤖.
Below, a word from its wisdom :

You are going to thank you later if it contains tests

@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

❌ Patch coverage is 42.10526% with 22 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (new-navigation@bd23353). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...components/deployment-action/deployment-action.tsx 22.22% 7 Missing ⚠️
...c/lib/components/action-toolbar/action-toolbar.tsx 0.00% 6 Missing ⚠️
...ervice-deployment-list/service-deployment-list.tsx 58.33% 4 Missing and 1 partial ⚠️
...nt-deployment-list/environment-deployment-list.tsx 70.00% 2 Missing and 1 partial ⚠️
...hooks/use-deployment-queue/use-deployment-queue.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             new-navigation    #2422   +/-   ##
=================================================
  Coverage                  ?   43.90%           
=================================================
  Files                     ?      969           
  Lines                     ?    18875           
  Branches                  ?     5521           
=================================================
  Hits                      ?     8287           
  Misses                    ?     9101           
  Partials                  ?     1487           
Flag Coverage Δ
unittests 43.90% <42.10%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- Introduced a `suspense` parameter to the `useDeploymentQueue` hook for improved loading state management.
- Updated the `ServiceDeploymentList` component to utilize the new `suspense` option in both `useService` and `useDeploymentHistory` hooks, enhancing user experience during data fetching.
…ify table cell styling

- Removed unnecessary padding and border classes from table header and cell components in both EnvironmentDeploymentList and ServiceDeploymentList for improved consistency and readability.
- Streamlined the mapping of header and cell elements by eliminating the index parameter where it was not needed.
@RemiBonnet RemiBonnet marked this pull request as ready for review February 26, 2026 07:39
@cursor
Copy link

cursor bot commented Feb 26, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 22.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

…e minWidth from table cell styles

- Eliminated minWidth property from header and cell components in both EnvironmentDeploymentList and ServiceDeploymentList to simplify styling and improve layout consistency.
Copy link
Contributor

@rmnbrd rmnbrd left a comment

Choose a reason for hiding this comment

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

Looks good to me 💯

@RemiBonnet RemiBonnet merged commit e946f61 into new-navigation Feb 26, 2026
10 checks passed
@RemiBonnet RemiBonnet deleted the feat/new-nav/service-deployment-history branch February 26, 2026 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants