Skip to content

Commit 35bb9de

Browse files
committed
refactor(app_config): Update AppConfigurationPage to use HtHttpException and friendly messages
1 parent bfe689e commit 35bb9de

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

lib/app_configuration/view/app_configuration_page.dart

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import 'package:ht_dashboard/l10n/l10n.dart';
55
import 'package:ht_dashboard/shared/constants/app_spacing.dart';
66
import 'package:ht_dashboard/shared/widgets/widgets.dart';
77
import 'package:ht_shared/ht_shared.dart';
8+
import 'package:ht_ui_kit/ht_ui_kit.dart'; // Import for toFriendlyMessage
89

910
/// {@template app_configuration_page}
1011
/// A page for managing the application's remote configuration.
@@ -80,15 +81,14 @@ class _AppConfigurationPageState extends State<AppConfigurationPage> {
8081
context.read<AppConfigurationBloc>().add(
8182
const AppConfigurationFieldChanged(),
8283
);
83-
} else if (state.status == AppConfigurationStatus.failure) {
84+
} else if (state.status == AppConfigurationStatus.failure &&
85+
state.exception != null) {
8486
ScaffoldMessenger.of(context)
8587
..hideCurrentSnackBar()
8688
..showSnackBar(
8789
SnackBar(
8890
content: Text(
89-
l10n.appConfigSaveErrorMessage(
90-
state.errorMessage ?? l10n.unknownError,
91-
),
91+
state.exception!.toFriendlyMessage(context),
9292
style: Theme.of(context).textTheme.bodyMedium?.copyWith(
9393
color: Theme.of(context).colorScheme.onError,
9494
),
@@ -108,8 +108,7 @@ class _AppConfigurationPageState extends State<AppConfigurationPage> {
108108
);
109109
} else if (state.status == AppConfigurationStatus.failure) {
110110
return FailureStateWidget(
111-
message:
112-
state.errorMessage ?? l10n.failedToLoadConfigurationMessage,
111+
exception: state.exception!,
113112
onRetry: () {
114113
context.read<AppConfigurationBloc>().add(
115114
const AppConfigurationLoaded(),

0 commit comments

Comments
 (0)