rfac,chor: bump dependencies and minor refactors

Signed-off-by: Aman <aman2@me.iitr.ac.in>

Author: aman-singh7

android/app/build.gradle

@@ -33,7 +33,7 @@ apply plugin: 'kotlin-android'
 apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
 
 android {
-    compileSdkVersion 32
+    compileSdkVersion 33
 
     compileOptions {
         sourceCompatibility JavaVersion.VERSION_1_8

lib/data/services/local/image_service.dart

@@ -34,6 +34,12 @@ class ImageService {
       cropStyle: CropStyle.circle,
       compressFormat: ImageCompressFormat.png,
     );
-    return croppedImage;
+
+    File? file;
+    if (croppedImage != null) {
+      file = File(croppedImage.path);
+    }
+
+    return file;
   }
 }

lib/data/services/local/storage_service.dart

@@ -103,6 +103,6 @@ class StorageService {
     }
   }
 
-  static set filter(ContestFilter? _filter) =>
-      _set(AppStrings.filterKey, json.encode(_filter!.toJson()));
+  static set filter(ContestFilter? filter) =>
+      _set(AppStrings.filterKey, json.encode(filter!.toJson()));
 }

lib/data/services/remote/notification_service.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: depend_on_referenced_packages
+
 import 'package:flutter_local_notifications/flutter_local_notifications.dart';
 import 'package:flutter_native_timezone/flutter_native_timezone.dart';
 import 'package:timezone/data/latest.dart' as tz;
@@ -13,20 +15,17 @@ class NotificationService {
         'channelName',
         importance: Importance.max,
       ),
-      iOS: IOSNotificationDetails(),
+      iOS: DarwinNotificationDetails(),
     );
   }
 
   static Future init() async {
     const settings = InitializationSettings(
       android: AndroidInitializationSettings('@mipmap/ic_launcher'),
-      iOS: IOSInitializationSettings(),
+      iOS: DarwinInitializationSettings(),
     );
 
-    await _notification.initialize(
-      settings,
-      onSelectNotification: (payload) {},
-    );
+    await _notification.initialize(settings);
 
     tz.initializeTimeZones();
     final locationName = await FlutterNativeTimezone.getLocalTimezone();

lib/domain/models/grouped_feed.dart

@@ -3,7 +3,7 @@ import 'package:freezed_annotation/freezed_annotation.dart';
 part 'grouped_feed.freezed.dart';
 part 'grouped_feed.g.dart';
 
-@freezed
+@Freezed(makeCollectionsUnmodifiable: false)
 class GroupedFeed with _$GroupedFeed {
   factory GroupedFeed({
     required String username,

lib/domain/models/sign_up.dart

@@ -1,4 +1,3 @@
-import 'package:flutter/foundation.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
 
 import 'handle.dart';

lib/domain/repositories/user_repository.dart

@@ -221,14 +221,14 @@ class UserRepository {
       headers: headers,
     );
 
-    final _users = <User>[];
+    final users = <User>[];
     if (response['status_code'] == 200) {
       for (final user in response['data'] ?? []) {
-        _users.add(User.fromJson(user));
+        users.add(User.fromJson(user));
       }
     }
 
-    return _users;
+    return users;
   }
 
   /// Get a list the names of recognized institutes.

lib/presentation/components/widgets/primary_button.dart

@@ -7,10 +7,12 @@ import '../../../data/constants/styles.dart';
 class PrimaryButton extends StatelessWidget {
   const PrimaryButton({
     required this.label,
+    this.isLoading = false,
     this.onPressed,
     Key? key,
   }) : super(key: key);
   final String label;
+  final bool isLoading;
   final VoidCallback? onPressed;
 
   @override
@@ -28,13 +30,21 @@ class PrimaryButton extends StatelessWidget {
         alignment: Alignment.center,
         height: 48.r,
         padding: EdgeInsets.symmetric(horizontal: 40.r),
-        child: Text(
-          label,
-          style: AppStyles.h6.copyWith(
-            color: AppColors.white,
-            fontWeight: FontWeight.w700,
-          ),
-        ),
+        child: isLoading
+            ? SizedBox(
+                width: 20.r,
+                height: 20.r,
+                child: const CircularProgressIndicator(
+                  color: AppColors.white,
+                ),
+              )
+            : Text(
+                label,
+                style: AppStyles.h6.copyWith(
+                  color: AppColors.white,
+                  fontWeight: FontWeight.w700,
+                ),
+              ),
       ),
     );
   }

lib/presentation/contests/bloc/contests_bloc.dart

@@ -159,33 +159,33 @@ extension on ContestFilter {
 
     if (!platfromCheck) return platfromCheck;
 
-    Duration _maxDuration;
+    Duration maxDuration;
     switch (duration) {
       case 0:
-        _maxDuration = const Duration(hours: 2);
+        maxDuration = const Duration(hours: 2);
         break;
       case 1:
-        _maxDuration = const Duration(hours: 3);
+        maxDuration = const Duration(hours: 3);
         break;
       case 2:
-        _maxDuration = const Duration(hours: 5);
+        maxDuration = const Duration(hours: 5);
         break;
       case 3:
-        _maxDuration = const Duration(days: 1);
+        maxDuration = const Duration(days: 1);
         break;
       case 4:
-        _maxDuration = const Duration(days: 10);
+        maxDuration = const Duration(days: 10);
         break;
       case 5:
-        _maxDuration = const Duration(days: 31);
+        maxDuration = const Duration(days: 31);
         break;
       default:
-        _maxDuration = Duration(days: 1e5.toInt());
+        maxDuration = Duration(days: 1e5.toInt());
     }
 
     final durationCheck = upcoming != null
-        ? upcoming.compareDuration(_maxDuration)
-        : ongoing!.compareDuration(_maxDuration);
+        ? upcoming.compareDuration(maxDuration)
+        : ongoing!.compareDuration(maxDuration);
 
     if (!durationCheck) return durationCheck;
 
@@ -200,17 +200,17 @@ extension on ContestFilter {
 }
 
 extension on Upcoming {
-  bool compareDuration(Duration _duration) {
-    return _duration.compareTo(endTime.difference(startTime)) >= 0;
+  bool compareDuration(Duration duration) {
+    return duration.compareTo(endTime.difference(startTime)) >= 0;
   }
 
-  bool compareStart(DateTime _startDate) {
-    return startTime.isAfter(_startDate);
+  bool compareStart(DateTime startDate) {
+    return startTime.isAfter(startDate);
   }
 }
 
 extension on Ongoing {
-  bool compareDuration(Duration _duration) {
-    return _duration.compareTo(endTime.difference(DateTime.now())) >= 0;
+  bool compareDuration(Duration duration) {
+    return duration.compareTo(endTime.difference(DateTime.now())) >= 0;
   }
 }

lib/presentation/contests/widgets/contest_card.dart

@@ -97,7 +97,7 @@ class ContestCard extends StatelessWidget {
               builder: (context, value, _) {
                 return IconButton(
                   onPressed: () {
-                    final _bloc = context.read<ContestsBloc>();
+                    final bloc = context.read<ContestsBloc>();
                     showDialog(
                       context: context,
                       builder: (context) {
@@ -208,7 +208,7 @@ class ContestCard extends StatelessWidget {
                               TextButton(
                                 onPressed: () {
                                   _notifier.value = false;
-                                  _bloc.pendingNotification
+                                  bloc.pendingNotification
                                       .remove(upcoming!.name);
 
                                   NotificationService.cancelNotification(
@@ -234,9 +234,9 @@ class ContestCard extends StatelessWidget {
                               onPressed: () {
                                 if (upcoming != null) {
                                   _notifier.value = true;
-                                  if (!_bloc.pendingNotification
+                                  if (!bloc.pendingNotification
                                       .contains(upcoming!.name)) {
-                                    _bloc.pendingNotification
+                                    bloc.pendingNotification
                                         .add(upcoming!.name);
                                   }
                                   NotificationService.setNotification(