Merge remote-tracking branch 'upstream/main' into fix/tags-and-start-date-clean

Author: mulikruchi07

lib/app/modules/home/controllers/home_controller.dart

@@ -550,15 +550,47 @@ class HomeController extends GetxController {
 
   isNeededtoSyncOnStart(BuildContext context) async {
     final SharedPreferences prefs = await SharedPreferences.getInstance();
-    bool? value;
-    value = prefs.getBool('sync-onStart') ?? false;
-    String? clientId, encryptionSecret;
-    clientId = await CredentialsStorage.getClientId();
-    encryptionSecret = await CredentialsStorage.getEncryptionSecret();
-    if (value) {
-      synchronize(context, false);
-      refreshTasks(clientId!, encryptionSecret!);
-    } else {}
+    final bool syncEnabled = prefs.getBool('sync-onStart') ?? false;
+    if (!syncEnabled) return;
+
+    final String? clientId = await CredentialsStorage.getClientId();
+    final String? encryptionSecret =
+        await CredentialsStorage.getEncryptionSecret();
+
+    try {
+      isRefreshing.value = true;
+      if (taskReplica.value) {
+        if (clientId != null && encryptionSecret != null) {
+          await refreshReplicaTasks();
+        }
+      } else if (taskchampion.value) {
+        if (clientId != null && encryptionSecret != null) {
+          await refreshTasks(clientId, encryptionSecret);
+        }
+      } else {
+        await synchronize(context, false);
+      }
+      if (context.mounted) {
+        final tColors =
+            Theme.of(context).extension<TaskwarriorColorTheme>()!;
+        ScaffoldMessenger.of(context).showSnackBar(
+          SnackBar(
+            content: Text(
+              'Sync Completed',
+              style: TextStyle(
+                color: tColors.primaryTextColor,
+              ),
+            ),
+            backgroundColor: tColors.primaryBackgroundColor,
+            duration: const Duration(seconds: 2),
+          ),
+        );
+      }
+    } catch (e) {
+      debugPrint('Error during sync on start: $e');
+    } finally {
+      isRefreshing.value = false;
+    }
   }
 
   RxBool syncOnStart = false.obs;

lib/app/modules/home/views/home_page_app_bar.dart

@@ -127,7 +127,7 @@ class HomePageAppBar extends StatelessWidget implements PreferredSizeWidget {
                 key: controller.refreshKey,
                 icon: !controller.isRefreshing.value
                     ? Icon(Icons.refresh, color: TaskWarriorColors.white)
-                    : Icon(Icons.autorenew, color: TaskWarriorColors.white),
+                    : const _SpinningIcon(),
                 onPressed: controller.isRefreshing.value
                     ? null
                     : () async {
@@ -151,8 +151,23 @@ class HomePageAppBar extends StatelessWidget implements PreferredSizeWidget {
                           }
                           debugPrint("Refreshing Replica tasks");
                           controller.isRefreshing.value = true;
-                          await controller.refreshReplicaTasks();
-                          controller.isRefreshing.value = false;
+                          try {
+                            await controller.refreshReplicaTasks();
+                            _showResultSnackBar(
+                                context, 'Sync Completed', false);
+                          } catch (e) {
+                            debugPrint('Error refreshing replica tasks: $e');
+                            _showResultSnackBar(
+                                context,
+                                SentenceManager(
+                                        currentLanguage:
+                                            controller.selectedLanguage.value)
+                                    .sentences
+                                    .homePageTaskWarriorNotConfigured,
+                                true);
+                          } finally {
+                            controller.isRefreshing.value = false;
+                          }
                           return;
                         }
 
@@ -177,13 +192,7 @@ class HomePageAppBar extends StatelessWidget implements PreferredSizeWidget {
                                   .hideCurrentSnackBar();
 
                               _showResultSnackBar(
-                                  context,
-                                  SentenceManager(
-                                          currentLanguage:
-                                              controller.selectedLanguage.value)
-                                      .sentences
-                                      .homePageFetchingTasks,
-                                  false);
+                                  context, 'Sync Completed', false);
                             } catch (e) {
                               debugPrint('Error refreshing tasks: $e');
                               ScaffoldMessenger.of(context)
@@ -216,6 +225,8 @@ class HomePageAppBar extends StatelessWidget implements PreferredSizeWidget {
                               await controller.synchronize(context, true);
                               ScaffoldMessenger.of(context)
                                   .hideCurrentSnackBar();
+                              _showResultSnackBar(
+                                  context, 'Sync Completed', false);
                             } catch (e) {
                               ScaffoldMessenger.of(context)
                                   .hideCurrentSnackBar();
@@ -280,3 +291,38 @@ class HomePageAppBar extends StatelessWidget implements PreferredSizeWidget {
   @override
   Size get preferredSize => AppBar().preferredSize;
 }
+
+class _SpinningIcon extends StatefulWidget {
+  const _SpinningIcon();
+
+  @override
+  State<_SpinningIcon> createState() => _SpinningIconState();
+}
+
+class _SpinningIconState extends State<_SpinningIcon>
+    with SingleTickerProviderStateMixin {
+  late final AnimationController _controller;
+
+  @override
+  void initState() {
+    super.initState();
+    _controller = AnimationController(
+      duration: const Duration(seconds: 1),
+      vsync: this,
+    )..repeat();
+  }
+
+  @override
+  void dispose() {
+    _controller.dispose();
+    super.dispose();
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return RotationTransition(
+      turns: _controller,
+      child: Icon(Icons.refresh, color: TaskWarriorColors.white),
+    );
+  }
+}