Skip to content

Expose in-app update checks and installed version - #2

Merged
Wirenut33 merged 1 commit into
mainfrom
codex/in-app-update-controls
Sep 5, 2026
Merged

Wirenut33 merged 1 commit into
mainfrom
codex/in-app-update-controls

Conversation

@Wirenut33

@Wirenut33 Wirenut33 commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Expose Check for Updates in QuickShot's application menu and Settings, in addition to the camera menu. Settings displays the installed version and explains automatic background updates. Sparkle's live state disables the action while a check is already running.

Validation: signed universal Apple Silicon/Intel build passed. The existing main release pipeline has now completed successfully, including encrypted-secret import, app/feed signing and GitHub Release publication.

Summary by CodeRabbit

  • New Features

    • Added “Check for Updates…” access to the application menu and Settings.
    • Added update status handling, including disabling the update action when unavailable.
    • Settings now displays the app name, installed version, update information, and automatic-update guidance.
  • Documentation

    • Updated automatic update documentation to reflect the new menu and Settings locations.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Essentials

Run ID: 78dea442-d256-4bf5-92fb-540cffb21147

📥 Commits

Reviewing files that changed from the base of the PR and between 607913f and 9b9046b.

📒 Files selected for processing (4)
  • QuickShot/QuickShotApp.swift
  • QuickShot/SettingsView.swift
  • QuickShot/UpdateControls.swift
  • README.md

Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Walkthrough

QuickShot adds Sparkle-backed update controls to the application menu and Settings view. Settings now shows the installed version, update action, and automatic-update guidance. The README documents both update locations.

Changes

Update access and presentation

Layer / File(s) Summary
Sparkle update control state
QuickShot/UpdateControls.swift
UpdateAvailability tracks Sparkle updater availability. CheckForUpdatesButton starts update checks and disables itself when unavailable.
Settings update presentation
QuickShot/SettingsView.swift
Settings displays the app version, update action, update information, and automatic-update guidance in a padded fixed-width layout.
Menu integration and documentation
QuickShot/QuickShotApp.swift, README.md
The application menu exposes CheckForUpdatesButton. The README documents the application menu and Settings locations.
Estimated code review effort: 2 (Simple) ~10 minutes

Merge Risk: ⚪ Minimal · up to 9b904

This change adds update-check entry points in the application menu and Settings, displays the installed version, and documents the new locations. No current merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main changes: exposing in-app update checks and displaying the installed version.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 3…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/in-app-update-controls

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

@Wirenut33
Wirenut33 merged commit ab135ba into main Sep 5, 2026
2 of 3 checks passed
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