feat(ui): improve provider update feedback - #1258
Conversation
|
Warning Review limit reachedNext included review available in 14 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (13)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Deploy Preview for devsydev canceled.
|
✅ Deploy Preview for images-devsy-sh canceled.
|
| {#if $providerVersions.refreshError} | ||
| <div class="mt-0.5 flex items-center gap-1 text-destructive"> | ||
| <AlertCircle class="size-3.5 shrink-0" /> | ||
| <span>Couldn't check for updates. Cached results may be out of date.</span> |
There was a problem hiding this comment.
change couldn't to Could not
|
@greptileai review |
|
|
@coderabbitai review |
|
| await providerRefreshState(provider.name) | ||
| providers.set(await providerList()) | ||
| updateError = null |
There was a problem hiding this comment.
If the post-update loadVersionsFor call fails, the sheet records a refresh failure. By then, the successful main-process job has already been removed, so providerRefreshState returns successfully without refreshing anything. This handler then reloads only the provider list and clears the error without retrying the failed version load, causing “Refresh status” to dismiss the warning while the displayed version data remains stale.
e2fa79a to
402c054
Compare
|
If you're new to commit signing, there are different ways to set it up: Sign commits with
|
402c054 to
56bbcf2
Compare
|
Tick the box to add this pull request to the merge queue (same as
|
Signed-off-by: Samuel K <skevetter@pm.me>
d22c713 to
d6c2744
Compare
Summary
Scope
Desktop provider update UX only. No CLI changes, page redesign, changelogs, update-all, auto-update, or provider-add changes.
Validation
npm run checknpm test(46 files, 423 tests)npm run electron:buildDraft for independent self-review. Do not trigger Greptile or CodeRabbit yet.