chore: misc

Author: fulleni

lib/account/bloc/available_countries_bloc.dart

@@ -28,7 +28,6 @@ class AvailableCountriesBloc
     }
     emit(state.copyWith(status: AvailableCountriesStatus.loading));
     try {
-      // TODO(fulleni): Add pagination if necessary for very large datasets.
       final response = await _countriesRepository.readAll(
         filter: {'hasActiveHeadlines': true},
         sort: [const SortOption('name', SortOrder.asc)],

lib/account/bloc/available_sources_bloc.dart

@@ -32,7 +32,6 @@ class AvailableSourcesBloc
     }
     emit(state.copyWith(status: AvailableSourcesStatus.loading));
     try {
-      // TODO(fulleni): Add pagination if necessary for very large datasets.
       // Filter to fetch only active sources.
       final response = await _sourcesRepository.readAll(
         filter: {'status': ContentStatus.active.name},

lib/account/bloc/available_topics_bloc.dart

@@ -28,7 +28,6 @@ class AvailableTopicsBloc
     }
     emit(state.copyWith(status: AvailableTopicsStatus.loading));
     try {
-      // TODO(fulleni): Add pagination if necessary for very large datasets.
       // Filter to fetch only active topics.
       final response = await _topicsRepository.readAll(
         filter: {'status': ContentStatus.active.name},

lib/entity_details/view/entity_details_page.dart

@@ -107,7 +107,8 @@ class _EntityDetailsViewState extends State<EntityDetailsView> {
     final colorScheme = theme.colorScheme;
     final remoteConfig = context.watch<AppBloc>().state.remoteConfig;
     final communityConfig = remoteConfig?.features.community;
-    final isSourceReportingEnabled = (communityConfig?.enabled ?? false) &&
+    final isSourceReportingEnabled =
+        (communityConfig?.enabled ?? false) &&
         (communityConfig?.reporting.sourceReportingEnabled ?? false);
 
     return Scaffold(
@@ -311,11 +312,11 @@ class _EntityDetailsViewState extends State<EntityDetailsView> {
                       },
                       itemBuilder: (BuildContext context) =>
                           <PopupMenuEntry<String>>[
-                        PopupMenuItem<String>(
-                          value: 'report',
-                          child: Text(l10n.reportActionLabel),
-                        ),
-                      ],
+                            PopupMenuItem<String>(
+                              value: 'report',
+                              child: Text(l10n.reportActionLabel),
+                            ),
+                          ],
                     )
                   else
                     const SizedBox.shrink(),

lib/router/router.dart

@@ -52,7 +52,6 @@ import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/ap
 import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/feed_settings_page.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/font_settings_page.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/language_settings_page.dart';
-import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/notification_settings_page.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/settings_page.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/settings/view/theme_settings_page.dart';
 import 'package:flutter_news_app_mobile_client_full_source_code/shared/services/content_limitation_service.dart';
@@ -303,12 +302,7 @@ GoRouter createRouter({
                     name: Routes.settingsFeedName,
                     builder: (context, state) => const FeedSettingsPage(),
                   ),
-                  GoRoute(
-                    path: Routes.settingsNotifications,
-                    name: Routes.settingsNotificationsName,
-                    builder: (context, state) =>
-                        const NotificationSettingsPage(),
-                  ),
+
                   GoRoute(
                     path: Routes.settingsLanguage,
                     name: Routes.settingsLanguageName,

lib/router/routes.dart

@@ -75,8 +75,6 @@ abstract final class Routes {
   static const settingsFeedName = 'settingsFeed';
   static const settingsArticle = 'article';
   static const settingsArticleName = 'settingsArticle';
-  static const settingsNotifications = 'notifications';
-  static const settingsNotificationsName = 'settingsNotifications';
   static const settingsLanguage = 'language';
   static const settingsLanguageName = 'settingsLanguage';