Skip to content

fix: use grouped apps endpoint on initial load, fall back to per-namespace polling#580

Merged
dcharles525 merged 4 commits into
1.14.0from
fix/initial-load-performance
May 20, 2026
Merged

fix: use grouped apps endpoint on initial load, fall back to per-namespace polling#580
dcharles525 merged 4 commits into
1.14.0from
fix/initial-load-performance

Conversation

@dcharles525
Copy link
Copy Markdown
Member

@dcharles525 dcharles525 commented May 19, 2026

PR Checklist

  • Linting Test is passing
  • Code is well documented
  • If applicable, a PR in the epinio/docs repository has been opened

Summary

Fixes the initial app list page performance issue where one API call was fired per namespace on load.

Pairs with epinio/epinio#3034.

Occurred changes and/or fixed issues

  • Added findGroupedApps store action that calls the new /api/v1/applications/grouped endpoint
  • onMounted now uses findGroupedApps for the initial load (1 call, server-side fan-out) instead of per-namespace calls
  • Per-namespace pagination and background polling continue using the existing findAppsInNamespace action unchanged
  • Increased polling interval to reduce steady-state load on the server

Technical notes summary

The grouped endpoint returns map[namespace]PaginatedResponse. The store action guards against epiniofy injecting non-namespace keys into the response map before classifying items.

Areas or cases that should be tested

  • App list page loads correctly across multiple namespaces
  • Per-namespace table pagination still works after initial load
  • Background polling still refreshes app state
  • A user with no namespace access sees an empty list, not an error

Areas which could experience regressions

  • App list page initial render
  • Per-namespace table pagination state

johnlcos and others added 2 commits May 19, 2026 15:53
… back to namesspace based polling. Increase polling time to lessen the load on the Epinio server):
@github-actions github-actions Bot added the kind/bug Something isn't working label May 19, 2026
@dcharles525 dcharles525 changed the title fix(Call the grouped app namespace endpoint on initial load then fall… fix: use grouped apps endpoint on initial load, fall back to per-namespace polling May 19, 2026
@dcharles525 dcharles525 marked this pull request as ready for review May 19, 2026 20:45
…b.com:epinio/ui into fix/initial-load-performance
Copy link
Copy Markdown
Collaborator

@johnlcos johnlcos left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@johnlcos johnlcos left a comment

Choose a reason for hiding this comment

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

LGTM

@dcharles525 dcharles525 merged commit 1515100 into 1.14.0 May 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants