Merge pull request #622 from flow-mn:develop

RC#2 0.16.6

Author: sadespresso

lib/routes/home_page.dart

@@ -246,7 +246,7 @@ class _HomePageState extends State<HomePage>
         throw "GoRouter is null";
       }
 
-      goRouter.go(path);
+      await goRouter.push(path);
 
       return true;
     } catch (e) {

lib/utils/extensions/go_router.dart

@@ -1,6 +1,10 @@
 // Ongoing issue about lack of `popUntil`
 // https://github.com/flutter/flutter/issues/131625
+import "package:flutter/widgets.dart";
 import "package:go_router/go_router.dart";
+import "package:logging/logging.dart";
+
+final Logger _log = Logger("GoRouterExt");
 
 extension GoRouterExt on GoRouter {
   void popUntil(bool Function(GoRoute) predicate) {
@@ -20,3 +24,29 @@ extension GoRouterExt on GoRouter {
     }
   }
 }
+
+extension GoRouterContextExt on BuildContext {
+  String get location {
+    final GoRouter router = GoRouter.of(this);
+
+    final RouteMatch lastMatch =
+        router.routerDelegate.currentConfiguration.last;
+    final RouteMatchList matchList = lastMatch is ImperativeRouteMatch
+        ? lastMatch.matches
+        : router.routerDelegate.currentConfiguration;
+    final String location = matchList.uri.toString();
+    return location;
+  }
+
+  void safePush(String path) {
+    try {
+      if (location != path) {
+        push(path);
+      } else {
+        _log.fine("Not navigating to the same path: $path");
+      }
+    } catch (e) {
+      push(path);
+    }
+  }
+}

pubspec.yaml

@@ -3,7 +3,7 @@ description: A personal finance managing app
 
 publish_to: "none" # Remove this line if you wish to publish to pub.dev
 
-version: "0.16.6+293"
+version: "0.16.6+294"
 
 environment:
     sdk: ">=3.10.0 <4.0.0"