Limit update notifications to once per startup

This logic was intentionally designed to continue to prompt the user to
update if they haven't, but that seems pretty anti-user. The change will
stop the update prompts from showing more than once per game startup,
unless manually invoked by the user a second time.

Closes https://github.com/ppy/osu/issues/13821.
This commit is contained in:
Dean Herbert 2021-07-09 18:23:27 +09:00
parent f3b68a4fbf
commit a9250a0d98

View File

@ -68,6 +68,8 @@ namespace osu.Desktop.Updater
return false; return false;
} }
scheduleRecheck = false;
if (notification == null) if (notification == null)
{ {
notification = new UpdateProgressNotification(this) { State = ProgressNotificationState.Active }; notification = new UpdateProgressNotification(this) { State = ProgressNotificationState.Active };
@ -98,7 +100,6 @@ namespace osu.Desktop.Updater
// could fail if deltas are unavailable for full update path (https://github.com/Squirrel/Squirrel.Windows/issues/959) // could fail if deltas are unavailable for full update path (https://github.com/Squirrel/Squirrel.Windows/issues/959)
// try again without deltas. // try again without deltas.
await checkForUpdateAsync(false, notification).ConfigureAwait(false); await checkForUpdateAsync(false, notification).ConfigureAwait(false);
scheduleRecheck = false;
} }
else else
{ {