Skip to content

Commit aae2924

Browse files
client: shorebird manager cleanup
1 parent aa803e8 commit aae2924

1 file changed

Lines changed: 10 additions & 20 deletions

File tree

school_data_hub_flutter/lib/core/updater/shorebird_update_manager.dart

Lines changed: 10 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import 'dart:async';
22

33
import 'package:flutter/foundation.dart';
4+
import 'package:flutter_it/flutter_it.dart';
45
import 'package:logging/logging.dart';
56
import 'package:school_data_hub_flutter/common/services/notification_service.dart';
67
import 'package:shorebird_code_push/shorebird_code_push.dart';
7-
import 'package:flutter_it/flutter_it.dart';
88

99
/// Update status with additional context
1010
enum UpdateManagerStatus {
@@ -78,19 +78,7 @@ class ShorebirdUpdateManager extends ChangeNotifier {
7878
_startAutoCheck();
7979
}
8080

81-
final updateAvailable = await checkForUpdates();
82-
if (updateAvailable) {
83-
di<NotificationService>().showSnackBar(
84-
NotificationType.info,
85-
'Ein Update wird heruntergeladen...',
86-
);
87-
_setStatus(UpdateManagerStatus.updateAvailable);
88-
await _updater.update(track: _currentTrack);
89-
90-
di<NotificationService>().showInformationDialog(
91-
'Ein Update ist verfügbar. Bitte installieren Sie es, um die neueste Version der App zu verwenden.',
92-
);
93-
}
81+
await checkForUpdates();
9482

9583
_log.info('ShorebirdUpdateManager initialized successfully');
9684
} catch (error, stackTrace) {
@@ -142,17 +130,13 @@ class ShorebirdUpdateManager extends ChangeNotifier {
142130
'Ein Update wird heruntergeladen...',
143131
);
144132
await _updater.update(track: _currentTrack);
145-
di<NotificationService>().showInformationDialog(
146-
'Ein Update wurde installiert. Bitte starten Sie die App neu, um die neueste Version der App zu verwenden.',
147-
);
133+
_showRestartRequiredDialog();
148134
_setStatus(UpdateManagerStatus.restartRequired);
149135
return true;
150136

151137
case UpdateStatus.restartRequired:
152138
_log.info('Restart required to apply update');
153-
di<NotificationService>().showInformationDialog(
154-
'Ein Update wurde installiert. Bitte starten Sie die App neu, um die neueste Version der App zu verwenden.',
155-
);
139+
_showRestartRequiredDialog();
156140
_setStatus(UpdateManagerStatus.restartRequired);
157141
return false;
158142

@@ -322,4 +306,10 @@ class ShorebirdUpdateManager extends ChangeNotifier {
322306
_autoCheckTimer?.cancel();
323307
_autoCheckTimer = null;
324308
}
309+
310+
void _showRestartRequiredDialog() {
311+
di<NotificationService>().showInformationDialog(
312+
'Ein Update wurde installiert. Bitte starten Sie die App neu, um die neueste Version der App zu verwenden.',
313+
);
314+
}
325315
}

0 commit comments

Comments
 (0)