chore(#74): Files which the freezed plugin generates should not be added to VCS (#78)

* chore(#74): Add the generated files to .gitignore

* chore(#74): Ignore nstack, router and injector configs

* nit

* chore(#74): Use pattern matching

* nit

* chore(#74): Generate files before building

* fix(#74): Build errors

Author: nivisi

.github/workflows/pull_request_validation.yml

@@ -13,6 +13,7 @@ jobs:
         with:
           channel: 'stable'
       - run: flutter pub get
+      - run: flutter pub run build_runner build --delete-conflicting-outputs
       - run: flutter build apk --release -t lib/main_production.dart
       - run: flutter analyze
       - run: flutter test

.gitignore

@@ -39,3 +39,10 @@ app.*.symbols
 
 # Obfuscation related
 app.*.map.json
+
+# Flutter code generation
+*.freezed.dart
+*.g.dart
+nstack.dart
+*.gr.dart
+*.config.dart

lib/data/services/response_error.dart

@@ -1,8 +1,6 @@
 import 'dart:io';
 
 import 'package:dio/dio.dart';
-import 'package:flutter/foundation.dart';
-import 'package:flutter/material.dart';
 import 'package:flutter_template/data/services/response_objects/error_response.dart';
 import 'package:flutter_template/nstack/nstack.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
@@ -16,6 +14,7 @@ part 'response_error.freezed.dart';
 @freezed
 class ResponseError<T> with _$ResponseError<T> implements Exception {
   const ResponseError._();
+
   const factory ResponseError.noInternetConnection() = _NoInternetConnection;
   const factory ResponseError.sendTimeout() = _SendTimeout;
   const factory ResponseError.connectTimeout() = _ConnectTimeout;
@@ -78,23 +77,23 @@ class ResponseError<T> with _$ResponseError<T> implements Exception {
           }
       }
     } else if (error is TypeError) {
-      debugPrint(error.toString());
+      // TODO: Log it
     }
     return const ResponseError.unexpectedError();
   }
 }
 
 extension ResponseErrorExtensions on ResponseError {
-  String getErrorMessage(BuildContext context) {
-    final _localization = context.localization.error;
+  String getErrorMessage(Localization l10n) {
+    final _localization = l10n.error;
 
     //TODO: create error module for errors and set value accordingly
     return when<String>(
       noInternetConnection: () => _localization.connectionError,
       sendTimeout: () => _localization.authenticationError,
       connectTimeout: () => _localization.authenticationError,
       receiveTimeout: () => _localization.authenticationError,
-      badRequest: (message) => message.getErrorMessage(context),
+      badRequest: (message) => message.getErrorMessage(l10n),
       notFound: () => _localization.authenticationError,
       tooManyRequests: () => _localization.authenticationError,
       unprocessableEntity: () => _localization.authenticationError,

lib/data/services/response_objects/error_response.dart

@@ -1,16 +1,14 @@
-import 'package:flutter/material.dart';
 import 'package:flutter_template/data/services/response_error.dart';
 import 'package:flutter_template/nstack/nstack.dart';
 import 'package:freezed_annotation/freezed_annotation.dart';
 
 part 'error_response.freezed.dart';
 part 'error_response.g.dart';
 
-@freezed
-
 ///The BE should provide one unique code for each error, in this case the
 ///error is being provided through errorName so we create an Enum for each
 ///code we are aware of with a default value to unknown
+@freezed
 class APIError with _$APIError {
   const factory APIError({
     required String? requestId,
@@ -55,8 +53,8 @@ extension ErrorResponseExtensions on ErrorResponse {
 }
 
 extension ErrorNameExtensions on ErrorName {
-  String getErrorMessage(BuildContext context) {
-    final _localization = context.localization.error;
+  String getErrorMessage(Localization l10n) {
+    final _localization = l10n.error;
 
     switch (this) {
       //Handle error enum and return mapped nstack vlaue

lib/injection/injector.config.dart

@@ -1,9 +1,9 @@
 import 'dart:async';
 
+import 'package:auto_route/auto_route.dart';
 import 'package:flutter/material.dart';
-import 'package:flutter_template/injection/injector.dart';
 import 'package:flutter_template/presentation/resources/resources.dart';
-import 'package:flutter_template/presentation/routes/app_navigator.dart';
+import 'package:flutter_template/presentation/routes/router.gr.dart';
 
 class SplashScreen extends StatefulWidget {
   const SplashScreen({Key? key}) : super(key: key);
@@ -18,7 +18,7 @@ class _SplashScreenState extends State<SplashScreen> {
   @override
   void initState() {
     _timer = Timer(const Duration(seconds: 2), () {
-      injector.get<AppNavigator>().navigateToHome();
+      context.router.navigate(HomeScreenRoute());
     });
     super.initState();
   }

lib/nstack/nstack.dart

@@ -1,13 +1,11 @@
 import 'package:auto_route/auto_route.dart';
 import 'package:flutter_template/presentation/feature/home/home_screen.dart';
 import 'package:flutter_template/presentation/feature/splash/splash_screen.dart';
-import 'package:injectable/injectable.dart';
 
 @MaterialAutoRouter(
   routes: <AutoRoute>[
     AutoRoute(page: SplashScreen, path: '/'),
     AutoRoute(page: HomeScreen, path: '/home'),
   ],
 )
-@Injectable()
 class $AppRouter {}