Skip to content

Conversation

@BP602
Copy link
Contributor

@BP602 BP602 commented Jul 7, 2025

This PR adds a user setting to control automatic update checking, addressing a real issue where auto-update errors can block user access to the application.

Problem

The auto-updater currently runs automatically on startup and can cause disruptive errors when builds aren't available for all platforms:

Error: Cannot find latest-linux.yml in the latest release artifacts
HttpError: 404 - https://github.com/KickTalkOrg/KickTalk/releases/download/1.1.3/latest-linux.yml

This error appears because the project doesn't currently have automated Linux/macOS build workflows, which is understandable for a project this size. However, the auto-update notification prevents users from accessing chats when it fails.

Screenshot From 2025-07-07 14-19-29

Solution

  • New setting: Added "Auto Update" toggle in General settings
  • Default behavior: Auto-updates remain enabled by default (no change for existing users)
  • User control: Users can disable automatic update checks to avoid blocking errors
  • Manual updates: Disabling auto-update still allows manual update checks through the UI
  • Settings persistence: Choice is saved and respected across app restarts

Screenshot From 2025-07-07 14-22-41

BP602 and others added 3 commits July 7, 2025 13:17
- Add cross-platform icon handling for Linux/macOS support
- Convert Windows .ico to .png for non-Windows platforms
- Update tray and thumbar icons to use platform-appropriate format

Fixes application crashes on Linux due to unsupported .ico format

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add auto-update toggle in General settings (enabled by default)
- Check settings before performing automatic update checks on startup
- Skip auto-updater when disabled in settings
- Add IPC handler for runtime setting changes
- Maintain existing manual update functionality

Users can now disable automatic update checking while keeping the ability to manually check for updates through the UI.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add immediate dismissal of update notifications when auto-update is disabled in settings
- No longer need to restart app to hide existing update notifications
- Add onDismiss method to update API in preload script
- Listen for autoUpdate setting changes in main process
- Update Updater component to handle dismiss events

Now when users disable auto-update in settings, any active update notifications disappear immediately instead of persisting until restart.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@BP602
Copy link
Contributor Author

BP602 commented Jul 7, 2025

Updated to handle case where if KickTalk was started with the auto-updater enabled and is showing a notification, and then the user goes into settings and disables auto-updates, the existing notification would still be there until restart. Now it dismisses.

@ftk789 ftk789 merged commit 47b74b0 into KickTalkOrg:main Aug 23, 2025
ftk789 pushed a commit that referenced this pull request Oct 14, 2025
…metry

Delay renderer telemetry until preload initialization
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.

2 participants