Merge pull request #882 from ImranR98/dev

Bugfix
This commit is contained in:
Imran Remtulla
2023-09-10 23:29:38 -04:00
committed by GitHub

View File

@@ -916,7 +916,7 @@ class AppsProvider with ChangeNotifier {
} }
} }
notifyListeners(); notifyListeners();
await exportApps(isAuto: true); exportApps(isAuto: true);
} }
Future<void> removeApps(List<String> appIds) async { Future<void> removeApps(List<String> appIds) async {
@@ -938,7 +938,7 @@ class AppsProvider with ChangeNotifier {
} }
if (appIds.isNotEmpty) { if (appIds.isNotEmpty) {
notifyListeners(); notifyListeners();
await exportApps(isAuto: true); exportApps(isAuto: true);
} }
} }
@@ -1100,6 +1100,9 @@ class AppsProvider with ChangeNotifier {
SettingsProvider settingsProvider = sp ?? this.settingsProvider; SettingsProvider settingsProvider = sp ?? this.settingsProvider;
var exportDir = await settingsProvider.getExportDir(); var exportDir = await settingsProvider.getExportDir();
if (isAuto) { if (isAuto) {
if (settingsProvider.autoExportOnChanges != true) {
return null;
}
if (exportDir == null) { if (exportDir == null) {
logs.add('Skipping auto-export as dir is not set.'); logs.add('Skipping auto-export as dir is not set.');
return null; return null;