Skip to content

Commit baf7d97

Browse files
committed
feat(app): add logger to AppBloc
- Added Logger to AppBloc - Improved repository names - Added doc comment to _AppView - Updated themeMode switch - Minor code style improvements
1 parent 6dafaab commit baf7d97

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

lib/app/view/app.dart

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import 'package:ht_dashboard/shared/theme/app_theme.dart';
1919
import 'package:ht_data_repository/ht_data_repository.dart';
2020
import 'package:ht_kv_storage_service/ht_kv_storage_service.dart';
2121
import 'package:ht_shared/ht_shared.dart' hide AppStatus;
22+
import 'package:logging/logging.dart';
2223

2324
class App extends StatelessWidget {
2425
const App({
@@ -78,11 +79,12 @@ class App extends StatelessWidget {
7879
BlocProvider(
7980
create: (context) => AppBloc(
8081
authenticationRepository: context.read<HtAuthRepository>(),
81-
userAppSettingsRepository: context
82-
.read<HtDataRepository<UserAppSettings>>(),
82+
userAppSettingsRepository:
83+
context.read<HtDataRepository<UserAppSettings>>(),
8384
appConfigRepository:
8485
context.read<HtDataRepository<RemoteConfig>>(),
8586
environment: _environment,
87+
logger: Logger('AppBloc'),
8688
),
8789
),
8890
BlocProvider(
@@ -92,7 +94,7 @@ class App extends StatelessWidget {
9294
),
9395
BlocProvider(
9496
create: (context) => AppConfigurationBloc(
95-
appConfigRepository:
97+
remoteConfigRepository:
9698
context.read<HtDataRepository<RemoteConfig>>(),
9799
),
98100
),
@@ -123,6 +125,7 @@ class App extends StatelessWidget {
123125
}
124126

125127
class _AppView extends StatefulWidget {
128+
/// {@macro app_view}
126129
const _AppView({
127130
required this.htAuthenticationRepository,
128131
required this.environment,
@@ -216,7 +219,7 @@ class _AppViewState extends State<_AppView> {
216219
themeMode: switch (baseTheme) {
217220
AppBaseTheme.light => ThemeMode.light,
218221
AppBaseTheme.dark => ThemeMode.dark,
219-
AppBaseTheme.system || null => ThemeMode.system,
222+
_ => ThemeMode.system,
220223
},
221224
locale: language != null ? Locale(language) : null,
222225
),

0 commit comments

Comments
 (0)