Skip to content

refactor: migrate UI project list and extensions to REST API#6294

Merged
rpelczar merged 1 commit into
akuity:mainfrom
rpelczar:refactor/api-migration-projects-list
May 27, 2026
Merged

refactor: migrate UI project list and extensions to REST API#6294
rpelczar merged 1 commit into
akuity:mainfrom
rpelczar:refactor/api-migration-projects-list

Conversation

@rpelczar

@rpelczar rpelczar commented May 14, 2026

Copy link
Copy Markdown
Contributor

fixes #6291

PR includes:

  • api migrations in the projects list and argocd ext
  • list projects endpoint refactor - added filters and pagination

@rpelczar rpelczar added this to the v1.11.0 milestone May 14, 2026
@rpelczar rpelczar self-assigned this May 14, 2026
@rpelczar rpelczar requested a review from a team as a code owner May 14, 2026 22:03
@rpelczar rpelczar added kind/refactor Non-functional changes to implementation details area/ui Affects the UI priority/normal This is the priority for most work area/api-server Affects Kargo's API server labels May 14, 2026
@netlify

netlify Bot commented May 14, 2026

Copy link
Copy Markdown

Deploy Preview for docs-kargo-io ready!

Name Link
🔨 Latest commit 3a68f8c
🔍 Latest deploy log https://app.netlify.com/projects/docs-kargo-io/deploys/6a0648d3dcdb300008356be4
😎 Deploy Preview https://deploy-preview-6294.docs.kargo.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rpelczar rpelczar force-pushed the refactor/api-migration-projects-list branch from 177d555 to f6585a6 Compare May 14, 2026 22:06
Extend the REST `listProjects` handler with `mine`, `filter`, `uid`,
`pageSize`, and `page` query params and a `{ items, total }` response
envelope so it matches the deprecated ConnectRPC contract. Port the UI
project list (and the supporting `useArgoCDURL` extension hook) off the
legacy ConnectRPC client to the regenerated REST hooks, dropping the
client-side pagination/filter shim and the obsolete protobuf-based
project cache.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Rafal Pelczar <rafal@akuity.io>
@rpelczar rpelczar force-pushed the refactor/api-migration-projects-list branch from f6585a6 to 3a68f8c Compare May 14, 2026 22:12
@codecov

codecov Bot commented May 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.98%. Comparing base (680a52e) to head (3a68f8c).

Files with missing lines Patch % Lines
pkg/server/list_projects_v1alpha1.go 92.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6294      +/-   ##
==========================================
+ Coverage   57.96%   57.98%   +0.01%     
==========================================
  Files         496      496              
  Lines       41602    41626      +24     
==========================================
+ Hits        24115    24137      +22     
- Misses      16033    16034       +1     
- Partials     1454     1455       +1     

☔ 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.

@Marvin9 Marvin9 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM Frontend and Backend. But just in case someone from backend team wants to skim through @fuskovic

@thomastaylor312 thomastaylor312 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good from the backend side!

@rpelczar rpelczar added this pull request to the merge queue May 27, 2026
Merged via the queue into akuity:main with commit 3d8df58 May 27, 2026
18 checks passed
@rpelczar rpelczar deleted the refactor/api-migration-projects-list branch May 27, 2026 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api-server Affects Kargo's API server area/ui Affects the UI kind/refactor Non-functional changes to implementation details priority/normal This is the priority for most work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI: API migration to REST - ext, projects list

3 participants