-
Notifications
You must be signed in to change notification settings - Fork 37
Enabling auto updates for macos #1380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces auto-update functionality for macOS builds, replacing the existing manual version checking system with an automated update mechanism that downloads and installs updates in the background.
Key changes:
- Replaces manual version notification system with automated updates for macOS
- Updates event listener API to support object-based configuration with optional enabling
- Adds new Tauri plugins for process management and updater functionality
Reviewed Changes
Copilot reviewed 10 out of 17 changed files in this pull request and generated 4 comments.
Show a summary per file
File | Description |
---|---|
gui/src/hooks/useUpdates.tsx | New hook that consolidates update logic, featuring auto-updates for macOS and manual notifications for other platforms |
gui/src/hooks/useEventListener.ts | Enhanced to accept object configuration with enabled flag and event parameter support |
gui/src/hooks/useNoticeNewVersion.tsx | Removed - functionality moved to useUpdates |
gui/src/routes/home/_l.tsx | Updated to use new useUpdates hook instead of useNoticeNewVersion |
crates/types/src/ui_events.rs | Added UpdateReady event type with version parameter |
Cargo.toml | Version downgrade from 1.20.2 to 1.10.2 |
gui/package.json | Added Tauri process and updater plugin dependencies |
Multiple transaction/connection files | Updated useEventListener calls to new object-based API |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
This is feature-gated behind the
updater
feature flag. We'll be testing this internally for now, and enable the flag in a future releaseIt only works for macos right now, because that's the only place where we have signing enabled so far. Hopefully linux appimages will soon follow
The way it works:
latest.json
that's already present in releases (since the previous one)