Skip to content
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

Add back missing app icon on Windows #7448

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

MarkusPettersson98
Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 commented Jan 10, 2025

Enable the signAndEditExecutable electron-builder option, which allows electron-builder to add metadata to the produced binary / app installer. Without this option enabled, the electron app was missing the Mullvad app icon. This option was disabled by accident, and the default value is also true.

Here are the relevant electron-builder docs: https://www.electron.build/win.html#signandeditexecutable


This change is Reviewable

Sign installer & add metadata to the produced binary / app installer.
Without this option enabled, the electron app was missing the Mullvad
app icon. This option was disabled by accident, and the default value is
also true.
Copy link

linear bot commented Jan 10, 2025

@MarkusPettersson98 MarkusPettersson98 merged commit 550ad64 into main Jan 10, 2025
10 checks passed
@MarkusPettersson98 MarkusPettersson98 deleted the desktop-icon-is-the-electron-logo-des-1605 branch January 10, 2025 13:41
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