fixup! TF-2965 Add retry capability for OIDC check request

linagora · Sep 16, 2024 · 4bdeb9c · 4bdeb9c
1 parent f5c740b
commit 4bdeb9c
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 4 deletions.
diff --git a/lib/features/login/presentation/base_login_view.dart b/lib/features/login/presentation/base_login_view.dart
@@ -55,7 +55,7 @@ abstract class BaseLoginView extends GetWidget<LoginController> {
         ),
         onPressed: onRetry,
         child: Text(
-          AppLocalizations.of(context).retry,
+          AppLocalizations.of(context).tryAgain,
           style: const TextStyle(fontSize: 16, color: Colors.white)
         )
       )

diff --git a/lib/features/login/presentation/widgets/login_message_widget.dart b/lib/features/login/presentation/widgets/login_message_widget.dart
@@ -11,6 +11,7 @@ import 'package:tmail_ui_user/features/login/domain/state/dns_lookup_to_get_jmap
 import 'package:tmail_ui_user/features/login/domain/state/get_oidc_configuration_state.dart';
 import 'package:tmail_ui_user/features/login/domain/state/get_token_oidc_state.dart';
 import 'package:tmail_ui_user/features/login/presentation/login_form_type.dart';
+import 'package:tmail_ui_user/main/exceptions/remote_exception.dart';
 import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
 import 'package:tmail_ui_user/main/routes/route_navigation.dart';
 import 'package:tmail_ui_user/main/utils/toast_manager.dart';
@@ -54,6 +55,10 @@ class LoginMessageWidget extends StatelessWidget {
               } else if (failure is GetTokenOIDCFailure && failure.exception is NoSuitableBrowserForOIDCException) {
                 return AppLocalizations.of(context).noSuitableBrowserForOIDC;
               } else if (failure is FeatureFailure) {
+                if (failure.exception is ConnectionError) {
+                  return AppLocalizations.of(context).youAreOffline;
+                }
+
                 return _toastManager?.getMessageByException(context, failure.exception)
                   ?? AppLocalizations.of(context).unknownError;
               } else {

diff --git a/lib/l10n/intl_messages.arb b/lib/l10n/intl_messages.arb
@@ -3993,5 +3993,17 @@
     "type": "text",
     "placeholders_order": [],
     "placeholders": {}
+  },
+  "tryAgain": "Try again",
+  "@tryAgain": {
+    "type": "text",
+    "placeholders_order": [],
+    "placeholders": {}
+  },
+  "youAreOffline": "You are offline. It looks like you are not connected.",
+  "@youAreOffline": {
+    "type": "text",
+    "placeholders_order": [],
+    "placeholders": {}
   }
 }
diff --git a/lib/main/localizations/app_localizations.dart b/lib/main/localizations/app_localizations.dart
@@ -4188,10 +4188,17 @@ class AppLocalizations {
     );
   }
 
-  String get retry {
+  String get tryAgain {
     return Intl.message(
-      'Retry',
-      name: 'retry',
+      'Try again',
+      name: 'tryAgain',
+    );
+  }
+
+  String get youAreOffline {
+    return Intl.message(
+      'You are offline. It looks like you are not connected.',
+      name: 'youAreOffline',
     );
   }
 }