proposal(14): electron-updater UX#8
Open
christian-byrne wants to merge 1 commit intoComfy-Org:mainfrom
Open
Conversation
- Add update channel support (stable/beta) with settings UI
- Enhance update banner with version comparison (v{current} → v{new})
- Add progress bar with download speed and ETA display
- Add release notes button that shows GitHub release body in modal
- Add beta channel badge for pre-release updates
- Enable generateUpdatesFilesForAllChannels in build config
- Add i18n keys for new update UI elements
- Document differential update status and rollback approaches
Amp-Thread-ID: https://ampcode.com/threads/T-019c7926-be08-704f-b583-1c5167315271
Kosinkadink
added a commit
that referenced
this pull request
Feb 22, 2026
Migrate renderer from 15 vanilla JS files to Vue 3 + Pinia + Tailwind CSS 4: - Typed IPC bridge (30+ methods) in src/preload/ - Pinia stores: sessionStore (running instances, terminals), installationStore - Promise-based modal system (useModal + ModalDialog) - 9 views and 7 reusable components with Composition API - vue-i18n runtime locale loading - Tailwind CSS 4 with design tokens extracted from old styles.css Code review fixes applied: - Fix runtime crash: use useI18n() in script setup (#1, #2) - Fix unsafe spread on non-array SettingsField values (#3) - Replace document.getElementById with reactive state in NewInstallModal (#4) - Use DetailSection component in DetailModal, removing 130 lines of duplication (#5) - Replace v-html/metaHtml with structured #meta slot and MetaPart arrays (#6) - Emit setting-updated from SettingField, SettingsView re-fetches on change (#7) - Eliminate prop mutations in DetailModal, use update:installation emit (#8) Amp-Thread-ID: https://ampcode.com/threads/T-019c840f-eaa4-771e-bb7f-a27118c0e3eb Co-authored-by: Amp <amp@ampcode.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposal 14: electron-updater UX Improvements
Summary
Improve the auto-update experience with update channels, rich progress UI, and release notes display — without changing the core update mechanism.
Changes
lib/updater.jsrenderer/update-banner.jsv{current} → v{new}), progress bar with speed/ETA, Release Notes button, beta channel badgerenderer/styles.csslocales/en.jsonpackage.jsongenerateUpdatesFilesForAllChannels: truein build configlib/ipc.jsupdateChannelselect field in Settings (stable/beta).github/proposals/proposal-updater-ux.mdWhat this does NOT change
Key tradeoffs
-betasuffix and mark as pre-release on GitHubProposal document
See
.github/proposals/proposal-updater-ux.mdfor the full analysis including current flow diagram, proposed architecture, and risk assessment.