Skip to content

[Fix] Show retry when recent pull requests fail - #3091

Merged
daniel-lxs merged 1 commit into
developfrom
fix/home-recent-pr-retry-3gjc5b0co4dvy
Sep 22, 2026
Merged

daniel-lxs merged 1 commit into
developfrom
fix/home-recent-pr-retry-3gjc5b0co4dvy

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Opened on behalf of @daniel-lxs. Follow up by mentioning @roomote-roomote, in the web UI, or in Telegram.

What changed

Home's Recent PRs panel now distinguishes an initial load failure from a successful empty result. It uses the shared retryable load state when no cached data exists, while keeping cached pull requests visible during a failed refetch.

Why this change was made

Previously, a failed Recent PRs request fell through to No recent pull requests. Yet., making an unavailable list look empty and leaving users without a recovery action.

Impact

Users can retry Recent PR loading in place without reloading Home or losing the existing All, Draft, and Ready filters. Authenticated desktop and 390 × 844 mobile proof covered request abort, the explicit failure state, request restoration, Retry, and the genuine successful empty response; no provider-side action was used.

Screenshots

Home Recent PRs load failure on desktop

Home Recent PRs recovered empty state on desktop

Home Recent PRs load failure on mobile

Home Recent PRs recovered empty state on mobile

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 22, 2026 05:32
@roomote-community

roomote-community Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed 4220c16

@daniel-lxs daniel-lxs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Verified initial-load failure and retry recovery to populated and empty results on the exact head. Cached rows remain usable through a failed background refresh, and filters and unsent Home input are preserved. Desktop/mobile inspection, six focused tests, web typecheck, scoped lint, and formatting passed.

@daniel-lxs
daniel-lxs merged commit a89aef6 into develop Sep 22, 2026
17 checks passed
@daniel-lxs
daniel-lxs deleted the fix/home-recent-pr-retry-3gjc5b0co4dvy branch September 22, 2026 19:45
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.

1 participant