Bug Fix

Author: Okanlawon Jamiu

README.md

@@ -7,6 +7,7 @@ App and Up Flutter Take Home Project
 This project is the implementation of the App and Up Flutter Take Home Project.
 
 ### Requirements
+
 -   Authentication with Firebase Authentication (Email + at least one social), design of the required screens is up to you.
 -   Show a list of books gotten from the Google Books API
 -   Allow the user to search for books
@@ -17,6 +18,21 @@ This project is the implementation of the App and Up Flutter Take Home Project.
 -   BONUS: handle App responsiveness on the Desktop/Web.
 
 ### Implementation
+
 The project was implemented with custom architecture similar to MVVM where
--   ViewModels are known as modifiers and
+-   ViewModels are known as notifiers
 -   Class Dependencies are managed using the Riverpod Providers
+-   There are usage of special providers such as StreamProvider, StateProviders that comes with riverpod
+-   It uses the classic repository approach
+-   Services for performing network requests, contextless navigation etc are grouped as well
+
+### Build Limitations
+
+-   The project depends on an API Key (for calling the books api) which I gitignored so kindly generate an APIKey and place an apiKey constant in lib/src/services/base/api_credentials.dart 
+-   Google Signin will also not work until you add a valid SHA 1 key.
+
+### When building for Web
+
+run with PORT 5000 for Google Signin to Work i.e flutter run -d chrome --web-port=5000
+
+### Find app screenshots here

lib/src/features/books/models/favorite_book.dart

@@ -11,18 +11,20 @@ class FavoriteBook {
   final List<String>? categories;
   final String? publisher;
   final String? publishedDate;
+  final String? userId;
 
   FavoriteBook({
     required this.id,
     required this.title,
     this.image,
     this.averageRatings,
     required this.maturityRating,
-    this.description = 'No Description',
+    this.description,
     this.authors,
     this.categories,
     this.publisher,
     this.publishedDate,
+    this.userId,
   });
 
   factory FavoriteBook.fromDocumentSnapshot(
@@ -38,6 +40,7 @@ class FavoriteBook {
         categories: List<String>.from(snapshot.data()?['categories'] ?? []),
         publishedDate: snapshot.data()?['publishedDate'],
         publisher: snapshot.data()?['publisher'],
+        userId: snapshot.data()?['userId'],
       );
 
   Map<String, dynamic> toMap() {

lib/src/features/books/notifiers/favorite_books_notifier.dart

@@ -5,10 +5,14 @@ import '../../../repositories/favorite_books_repository.dart';
 import '../models/favorite_book.dart';
 
 class FavoriteBooksNotitier extends BaseChangeNotifier {
-  FavoriteBooksNotitier(this._read);
+  FavoriteBooksNotitier(this._read, {required this.bookId}) {
+    checkIsFavorite(bookId);
+  }
 
   final Reader _read;
 
+  final String bookId;
+
   bool _isFavorite = false;
   bool get isFavorite => _isFavorite;
 
@@ -38,6 +42,7 @@ class FavoriteBooksNotitier extends BaseChangeNotifier {
   }
 }
 
-final favoriteBooksNotifierProvider = ChangeNotifierProvider(
-  (ref) => FavoriteBooksNotitier(ref.read),
+final favoriteBooksNotifierProvider =
+    ChangeNotifierProvider.family<FavoriteBooksNotitier, String>(
+  (ref, bookId) => FavoriteBooksNotitier(ref.read, bookId: bookId),
 );

lib/src/features/books/providers/favorite_books_provider.dart

@@ -2,6 +2,6 @@ import 'package:hooks_riverpod/hooks_riverpod.dart';
 
 import '../../../repositories/favorite_books_repository.dart';
 
-final favoriteBooksProvider = StreamProvider((ref) {
+final favoriteBooksProvider = StreamProvider.autoDispose((ref) {
   return ref.watch(favoriteBooksRepository).getFavoriteBooks();
 });

lib/src/features/books/views/book_details_view.dart

@@ -31,16 +31,15 @@ class BookDetailsView extends StatelessWidget {
         actions: [
           Consumer(
             builder: (context, watch, child) {
-              final notifier = watch(favoriteBooksNotifierProvider)
-                ..checkIsFavorite(book.id);
+              final notifier = watch(favoriteBooksNotifierProvider(book.id));
 
               return IconButton(
                 onPressed: () {
                   final FavoriteBook fBook =
                       ModelConverter.toFavoriteBook(book);
 
                   context
-                      .read(favoriteBooksNotifierProvider)
+                      .read(favoriteBooksNotifierProvider(book.id))
                       .addOrRemoveFromFavorite(fBook);
                 },
                 icon: notifier.isFavorite

lib/src/features/books/views/favorite_books_view.dart

@@ -193,7 +193,7 @@ class BookListItem extends StatelessWidget {
                     IconButton(
                       onPressed: () {
                         context
-                            .read(favoriteBooksNotifierProvider)
+                            .read(favoriteBooksNotifierProvider(book.id))
                             .addOrRemoveFromFavorite(book);
                       },
                       icon: const Icon(Icons.delete),

lib/src/features/profile/notifiers/profile_notifier.dart

@@ -23,6 +23,6 @@ class ProfileNotifier extends BaseChangeNotifier {
   }
 }
 
-final profileNotifierProvider = ChangeNotifierProvider(
+final profileNotifierProvider = ChangeNotifierProvider.autoDispose(
   (ref) => ProfileNotifier(ref.read),
 );

lib/src/features/profile/notifiers/update_email_notifier.dart

@@ -48,6 +48,6 @@ class UpdateEmailNotifier extends BaseChangeNotifier {
   }
 }
 
-final updateEmailNotifierProvider = ChangeNotifierProvider(
+final updateEmailNotifierProvider = ChangeNotifierProvider.autoDispose(
   (ref) => UpdateEmailNotifier(ref.read),
 );

lib/src/features/profile/notifiers/update_password_notifier.dart

@@ -51,6 +51,6 @@ class UpdatePasswordNotifier extends BaseChangeNotifier {
   }
 }
 
-final updatePasswordNotifierProvider = ChangeNotifierProvider(
+final updatePasswordNotifierProvider = ChangeNotifierProvider.autoDispose(
   (ref) => UpdatePasswordNotifier(ref.read),
 );

lib/src/features/profile/notifiers/update_profile_notifier.dart

@@ -39,6 +39,6 @@ class UpdateProfileNotifier extends BaseChangeNotifier {
   }
 }
 
-final updateProfileNotifierProvider = ChangeNotifierProvider(
+final updateProfileNotifierProvider = ChangeNotifierProvider.autoDispose(
   (ref) => UpdateProfileNotifier(ref.read),
 );