Merge pull request #28 from headlines-toolkit/debug_integrate_proper_exceptions_l10n_messages

Debug integrate proper exceptions l10n messages

Author: fulleni

README.md

@@ -35,6 +35,7 @@ Control the behavior and appearance of the `ht_main` mobile application remotely
 #### 📊 **Intuitive User Interface**
 Built with Flutter, the dashboard provides a responsive and user-friendly
 experience across various web browsers and screen sizes.
+*   **User-Friendly Error Handling:** Displays clear, localized error messages for a smooth and understandable user experience when issues arise.
 *   **Benefit for you:** A modern, maintainable, and visually appealing
     interface for your administrative tasks. ✨
 

analysis_options.yaml

@@ -1,5 +1,6 @@
 analyzer:
   errors:
+    avoid_bool_literals_in_conditional_expressions: ignore
     avoid_catches_without_on_clauses: ignore
     avoid_print: ignore
     avoid_redundant_argument_values: ignore

lib/app/view/app.dart

@@ -19,6 +19,7 @@ import 'package:ht_dashboard/shared/theme/app_theme.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
 import 'package:ht_kv_storage_service/ht_kv_storage_service.dart';
 import 'package:ht_shared/ht_shared.dart' hide AppStatus;
+import 'package:ht_ui_kit/ht_ui_kit.dart';
 import 'package:logging/logging.dart';
 
 class App extends StatelessWidget {
@@ -29,7 +30,8 @@ class App extends StatelessWidget {
     required HtDataRepository<Country> htCountriesRepository,
     required HtDataRepository<Source> htSourcesRepository,
     required HtDataRepository<UserAppSettings> htUserAppSettingsRepository,
-    required HtDataRepository<UserContentPreferences> htUserContentPreferencesRepository,
+    required HtDataRepository<UserContentPreferences>
+    htUserContentPreferencesRepository,
     required HtDataRepository<RemoteConfig> htRemoteConfigRepository,
     required HtDataRepository<DashboardSummary> htDashboardSummaryRepository,
     required HtKVStorageService kvStorageService,
@@ -53,7 +55,8 @@ class App extends StatelessWidget {
   final HtDataRepository<Country> _htCountriesRepository;
   final HtDataRepository<Source> _htSourcesRepository;
   final HtDataRepository<UserAppSettings> _htUserAppSettingsRepository;
-  final HtDataRepository<UserContentPreferences> _htUserContentPreferencesRepository;
+  final HtDataRepository<UserContentPreferences>
+  _htUserContentPreferencesRepository;
   final HtDataRepository<RemoteConfig> _htRemoteConfigRepository;
   final HtDataRepository<DashboardSummary> _htDashboardSummaryRepository;
   final HtKVStorageService _kvStorageService;
@@ -79,10 +82,10 @@ class App extends StatelessWidget {
           BlocProvider(
             create: (context) => AppBloc(
               authenticationRepository: context.read<HtAuthRepository>(),
-              userAppSettingsRepository:
-                  context.read<HtDataRepository<UserAppSettings>>(),
-              appConfigRepository:
-                  context.read<HtDataRepository<RemoteConfig>>(),
+              userAppSettingsRepository: context
+                  .read<HtDataRepository<UserAppSettings>>(),
+              appConfigRepository: context
+                  .read<HtDataRepository<RemoteConfig>>(),
               environment: _environment,
               logger: Logger('AppBloc'),
             ),
@@ -94,8 +97,8 @@ class App extends StatelessWidget {
           ),
           BlocProvider(
             create: (context) => AppConfigurationBloc(
-              remoteConfigRepository:
-                  context.read<HtDataRepository<RemoteConfig>>(),
+              remoteConfigRepository: context
+                  .read<HtDataRepository<RemoteConfig>>(),
             ),
           ),
           BlocProvider(
@@ -107,10 +110,10 @@ class App extends StatelessWidget {
           ),
           BlocProvider(
             create: (context) => DashboardBloc(
-              dashboardSummaryRepository:
-                  context.read<HtDataRepository<DashboardSummary>>(),
-              appConfigRepository:
-                  context.read<HtDataRepository<RemoteConfig>>(),
+              dashboardSummaryRepository: context
+                  .read<HtDataRepository<DashboardSummary>>(),
+              appConfigRepository: context
+                  .read<HtDataRepository<RemoteConfig>>(),
               headlinesRepository: context.read<HtDataRepository<Headline>>(),
             ),
           ),
@@ -209,9 +212,11 @@ class _AppViewState extends State<_AppView> {
                 child: MaterialApp.router(
                   debugShowCheckedModeBanner: false,
                   routerConfig: _router,
-                  localizationsDelegates:
-                      AppLocalizations.localizationsDelegates,
-                  supportedLocales: AppLocalizations.supportedLocales,
+                  localizationsDelegates: const [
+                    HtUiKitLocalizations.delegate,
+                    ...AppLocalizations.localizationsDelegates,
+                  ],
+                  supportedLocales: HtUiKitLocalizations.supportedLocales,
                   theme: baseTheme == AppBaseTheme.dark
                       ? darkThemeData
                       : lightThemeData,

lib/app_configuration/bloc/app_configuration_bloc.dart

@@ -43,14 +43,14 @@ class AppConfigurationBloc
       emit(
         state.copyWith(
           status: AppConfigurationStatus.failure,
-          errorMessage: e.message,
+          exception: e,
         ),
       );
     } catch (e) {
       emit(
         state.copyWith(
           status: AppConfigurationStatus.failure,
-          errorMessage: 'An unknown error occurred: $e',
+          exception: UnknownException('An unknown error occurred: $e'),
         ),
       );
     }
@@ -79,14 +79,14 @@ class AppConfigurationBloc
       emit(
         state.copyWith(
           status: AppConfigurationStatus.failure,
-          errorMessage: e.message,
+          exception: e,
         ),
       );
     } catch (e) {
       emit(
         state.copyWith(
           status: AppConfigurationStatus.failure,
-          errorMessage: 'An unknown error occurred: $e',
+          exception: UnknownException('An unknown error occurred: $e'),
         ),
       );
     }

lib/app_configuration/bloc/app_configuration_event.dart

@@ -20,7 +20,7 @@ class AppConfigurationLoaded extends AppConfigurationEvent {
 /// {@template app_configuration_updated}
 /// Event to request the update of the application configuration.
 ///
-/// Carries the new [appConfig] object to be saved.
+/// Carries the new "appConfig" object to be saved.
 /// {@endtemplate}
 class AppConfigurationUpdated extends AppConfigurationEvent {
   /// {@macro app_configuration_updated}

lib/app_configuration/bloc/app_configuration_state.dart

@@ -24,7 +24,7 @@ class AppConfigurationState extends Equatable {
     this.status = AppConfigurationStatus.initial,
     this.remoteConfig,
     this.originalRemoteConfig,
-    this.errorMessage,
+    this.exception,
     this.isDirty = false,
     this.showSaveSuccess = false,
   });
@@ -38,8 +38,8 @@ class AppConfigurationState extends Equatable {
   /// The original application configuration loaded from the backend.
   final RemoteConfig? originalRemoteConfig;
 
-  /// An error message if an operation failed.
-  final String? errorMessage;
+  /// An error exception if an operation failed.
+  final HtHttpException? exception;
 
   /// Indicates if there are unsaved changes to the configuration.
   final bool isDirty;
@@ -52,7 +52,7 @@ class AppConfigurationState extends Equatable {
     AppConfigurationStatus? status,
     RemoteConfig? remoteConfig,
     RemoteConfig? originalRemoteConfig,
-    String? errorMessage,
+    HtHttpException? exception,
     bool? isDirty,
     bool clearErrorMessage = false,
     bool? showSaveSuccess,
@@ -62,9 +62,7 @@ class AppConfigurationState extends Equatable {
       status: status ?? this.status,
       remoteConfig: remoteConfig ?? this.remoteConfig,
       originalRemoteConfig: originalRemoteConfig ?? this.originalRemoteConfig,
-      errorMessage: clearErrorMessage
-          ? null
-          : errorMessage ?? this.errorMessage,
+      exception: clearErrorMessage ? null : exception ?? this.exception,
       isDirty: isDirty ?? this.isDirty,
       showSaveSuccess: clearShowSaveSuccess
           ? false
@@ -77,7 +75,7 @@ class AppConfigurationState extends Equatable {
     status,
     remoteConfig,
     originalRemoteConfig,
-    errorMessage,
+    exception,
     isDirty,
     showSaveSuccess,
   ];