Conversation
… in GUI Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com> Agent-Logs-Url: https://github.com/Tyrrrz/DiscordChatExporter/sessions/158dba86-9958-4f57-ab22-174e0606b42f
Tyrrrz
reviewed
Mar 21, 2026
…CORDCHATEXPORTER_ALLOW_AUTO_UPDATE Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com> Agent-Logs-Url: https://github.com/Tyrrrz/DiscordChatExporter/sessions/aa3944a4-e542-4770-8d17-2acb2c958ce9
Contributor
Author
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Copilot
AI
changed the title
Add DISCORDCHATEXPORTER_DISABLE_UPDATE env var to suppress auto-update in GUI
Add DISCORDCHATEXPORTER_ALLOW_AUTO_UPDATE env var to suppress auto-update in GUI
Mar 21, 2026
Tyrrrz
reviewed
Mar 21, 2026
…y accept false Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com> Agent-Logs-Url: https://github.com/Tyrrrz/DiscordChatExporter/sessions/1467eee2-3069-4f0b-abdf-b1dc65e00a9a
Contributor
Author
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
DISCORDCHATEXPORTER_ALLOW_AUTO_UPDATE env var to suppress auto-update in GUI
Owner
|
@copilot formatting issues |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a startup-level switch to fully disable the GUI auto-update system for package-based deployments by introducing an environment variable gate that affects both the update infrastructure and the Settings UI.
Changes:
- Added
StartOptions.IsAutoUpdateAllowed, controlled byDISCORDCHATEXPORTER_ALLOW_AUTO_UPDATE. - Prevented
UpdateManagercreation when auto-updates are disallowed (in addition to non-Windows short-circuiting). - Hid the auto-update Settings UI panel when auto-updates aren’t available.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| DiscordChatExporter.Gui/StartOptions.cs | Introduces IsAutoUpdateAllowed and reads DISCORDCHATEXPORTER_ALLOW_AUTO_UPDATE to control update availability. |
| DiscordChatExporter.Gui/Services/UpdateService.cs | Gates UpdateManager instantiation on Windows + IsAutoUpdateAllowed. |
| DiscordChatExporter.Gui/ViewModels/Dialogs/SettingsViewModel.cs | Exposes IsAutoUpdateAvailable for UI binding (Windows + env var gate). |
| DiscordChatExporter.Gui/Views/Dialogs/SettingsView.axaml | Switches visibility logic for the auto-update panel to bind to IsAutoUpdateAvailable. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com> Agent-Logs-Url: https://github.com/Tyrrrz/DiscordChatExporter/sessions/7933cfe4-ecad-4697-8b9c-ee3991aa147e
Contributor
Author
Fixed in a1f16df — moved Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
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
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.
IsAutoUpdateAllowedproperty toStartOptions.csreading fromDISCORDCHATEXPORTER_ALLOW_AUTO_UPDATEenv varUpdateService.csto useIsAutoUpdateAllowedIsAutoUpdateAvailableproperty toSettingsViewModel.csusingIsAutoUpdateAllowedSettingsView.axamlto bindIsVisibleon the auto-update panel toIsAutoUpdateAvailableIsAutoUpdateAllowedexpression: grouped in!(), inverted to positiveis { } envpattern, renamedv→env, onlyfalseaccepted (removed0)is { } envmoved to indented new line per formatter expectation✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.