tweak: still show main screen after registration failure

get10101 · Jun 12, 2024 · b7e8788 · b7e8788
1 parent dc9a5f4
commit b7e8788
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 10 deletions.
diff --git a/mobile/lib/features/welcome/loading_screen.dart b/mobile/lib/features/welcome/loading_screen.dart
@@ -36,6 +36,8 @@ class _LoadingScreenState extends State<LoadingScreen> {
   }
 
   Future<void> initAsync() async {
+    bool failedRegistration = false;
+
     try {
       await widget.task?.future;
     } catch (error, stackTrace) {
@@ -54,8 +56,7 @@ class _LoadingScreenState extends State<LoadingScreen> {
                 ),
               ]));
 
-      GoRouter.of(rootNavigatorKey.currentContext!).go(Onboarding.route);
-      return;
+      failedRegistration = true;
     }
 
     final [position, seedPresent, backupRequired, registeredForBeta] = await Future.wait<dynamic>([
@@ -67,11 +68,29 @@ class _LoadingScreenState extends State<LoadingScreen> {
     FlutterNativeSplash.remove();
 
     if (seedPresent) {
-      if (!registeredForBeta) {
+      if (!failedRegistration && !registeredForBeta) {
         logger.w("Registering for beta program despite having a seed; "
             "onboarding flow was probably previously interrupted");
         setState(() => message = "Registering for beta program");
-        await resumeRegisterForBeta();
+
+        try {
+          await resumeRegisterForBeta();
+        } catch (error, stackTrace) {
+          logger.e(registerErrorTitle, error: error, stackTrace: stackTrace);
+          await showDialog(
+              context: rootNavigatorKey.currentContext!,
+              builder: (context) => AlertDialog(
+                      title: const Text(registerErrorTitle),
+                      content: const Text(registerErrorBody),
+                      actions: [
+                        TextButton(
+                          onPressed: () {
+                            Navigator.pop(context, 'OK');
+                          },
+                          child: const Text('OK'),
+                        ),
+                      ]));
+        }
       }
 
       if (backupRequired) {

diff --git a/mobile/lib/features/welcome/welcome_screen.dart b/mobile/lib/features/welcome/welcome_screen.dart
@@ -208,12 +208,8 @@ class _WelcomeScreenState extends State<WelcomeScreen> {
                                             _formKey.currentState!.validate()) {
                                           final task = LoadingScreenTask(
                                               future: setupWallet(),
-                                              errorTitle: "Failed to register",
-                                              errorHelp: "Failed to register "
-                                                  "for the beta program.\n"
-                                                  "Check your internet connection. "
-                                                  "Otherwise, 10101 services may "
-                                                  "be down for maintenance.");
+                                              errorTitle: registerErrorTitle,
+                                              errorHelp: registerErrorBody);
                                           GoRouter.of(context).go(LoadingScreen.route, extra: task);
                                         }
                                       },
@@ -284,6 +280,11 @@ class _WelcomeScreenState extends State<WelcomeScreen> {
   }
 }
 
+const String registerErrorTitle = "Failed to register";
+const String registerErrorBody = "Failed to register for the beta program.\n"
+    "Check your internet connection. Otherwise, 10101 services may "
+    "be down for maintenance.";
+
 /// Resume a previously incomplete onboarding flow by registering for beta
 Future<void> resumeRegisterForBeta() async {
   await api.registerBeta(