Skip to content

fix(desktop): hide redundant 'latest version' status in settings#463

Merged
skevetter merged 1 commit into
mainfrom
desktop/simplify-version-settings
May 30, 2026
Merged

fix(desktop): hide redundant 'latest version' status in settings#463
skevetter merged 1 commit into
mainfrom
desktop/simplify-version-settings

Conversation

@skevetter

@skevetter skevetter commented May 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the inline "You're on the latest version" status block from the Updates section of the desktop settings page, which was redundant alongside the version display and Check for Updates button.
  • Inline status still appears for actionable states (checking, available, downloading, downloaded, error); the up-to-date case is now silent.

Summary by CodeRabbit

  • Bug Fixes
    • Improved version status section visibility to display only when relevant, preventing unnecessary UI elements from appearing when no update information is available.

Review Change Stack

@netlify

netlify Bot commented May 30, 2026

Copy link
Copy Markdown

Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit b7e1e71
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a1a41c27a28770008f05ecf

@coderabbitai

coderabbitai Bot commented May 30, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ccd3f1fe-fd05-4108-b476-92a0ed6a0b6d

📥 Commits

Reviewing files that changed from the base of the PR and between 30e2060 and b7e1e71.

📒 Files selected for processing (1)
  • desktop/src/renderer/src/lib/components/update/UpdatesSection.svelte

📝 Walkthrough

Walkthrough

The Version status section visibility is refined to hide the panel when the update state is "not-available", in addition to the existing exclusion for the "idle" state. The conditional rendering check is tightened by one line.

Changes

Update Status Panel Visibility

Layer / File(s) Summary
Tighten status panel visibility condition
desktop/src/renderer/src/lib/components/update/UpdatesSection.svelte
The conditional controlling whether the version status section renders is updated to exclude liveStatus.state === "not-available" in addition to the existing "idle" check, preventing the panel from displaying when the update availability cannot be determined.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

size/s

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@skevetter
skevetter marked this pull request as ready for review May 30, 2026 09:12
@skevetter
skevetter merged commit fafc8ca into main May 30, 2026
21 checks passed
@skevetter
skevetter deleted the desktop/simplify-version-settings branch May 30, 2026 09:13
@coderabbitai coderabbitai Bot added the size/s label May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant