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

Implement changes to suggestedUpgrade in Electron #7881

Conversation

tobias-jarvelov
Copy link
Contributor

@tobias-jarvelov tobias-jarvelov commented Mar 25, 2025

When the AppVersionInfo.suggestedUpgrade property is updated, due to the changes in DES-1811 we need to make some changes in the code to handle the new format.


This change is Reviewable

Copy link

linear bot commented Mar 25, 2025

@tobias-jarvelov tobias-jarvelov force-pushed the handle-changes-to-suggestedupgrade-format-des-1932 branch 2 times, most recently from 98054e1 to 103604f Compare March 25, 2025 11:26
@tobias-jarvelov tobias-jarvelov changed the title Handle changes to suggestedupgrade in Electron Implement changes to suggestedupgrade in Electron Mar 25, 2025
@tobias-jarvelov tobias-jarvelov force-pushed the handle-changes-to-suggestedupgrade-format-des-1932 branch 2 times, most recently from 9d17706 to b77ff14 Compare March 25, 2025 11:38
@tobias-jarvelov tobias-jarvelov requested a review from olmoh March 25, 2025 11:39
@tobias-jarvelov tobias-jarvelov changed the title Implement changes to suggestedupgrade in Electron Implement changes to suggestedUpgrade in Electron Mar 25, 2025
@dlon dlon force-pushed the in-app-upgrades-on-windows-and-macos branch from 9ac60e2 to 268a41c Compare March 25, 2025 12:35
It is never used and therefore does not need it.
To reflect changes in management interface where
suggestedUpgrade has changed from a string to a
message with its own fields.
@tobias-jarvelov tobias-jarvelov force-pushed the handle-changes-to-suggestedupgrade-format-des-1932 branch from b77ff14 to b1d279e Compare March 25, 2025 13:03
@tobias-jarvelov tobias-jarvelov force-pushed the handle-changes-to-suggestedupgrade-format-des-1932 branch from b1d279e to 1ba0f9e Compare March 25, 2025 13:15
@tobias-jarvelov tobias-jarvelov requested a review from olmoh March 25, 2025 14:37
@tobias-jarvelov tobias-jarvelov merged commit 114b384 into in-app-upgrades-on-windows-and-macos Mar 25, 2025
9 checks passed
@tobias-jarvelov tobias-jarvelov deleted the handle-changes-to-suggestedupgrade-format-des-1932 branch March 25, 2025 14:52
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