Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update dependencies and refactor code for improved performance… #136

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

berhili098
Copy link

This pull request includes several updates across different parts of the codebase, focusing on refactoring and improvements in the notifier classes, database configuration, and routing. The most important changes are summarized below:

Notifier Updates:

  • Updated hash generation for CurrentAppThemeNotifier, DownloadsNotifier, FavoritesNotifier, and BookDetailsNotifier to new values. [1] [2] [3] [4]
  • Modified ignore_for_file comments to include additional lint rules in current_app_theme_notifier.g.dart, downloads_notifier.g.dart, and favorites_notifier.g.dart. [1] [2] [3]

Database Configuration:

  • Removed the unused import package:sembast/sembast.dart from database_config.dart.
  • Refactored the init method in DatabaseConfig to improve readability.

Routing Adjustments:

  • Refactored the AppRouter class to extend RootStackRouter and updated the generated routes in app_router.gr.dart. [1] [2] [3] [4] [5]

UI Improvements:

  • Simplified string replacement logic in fileName getter in download_alert.dart.
  • Refactored createFile method for better readability in download_alert.dart.
  • Updated onPopInvoked method to onPopInvokedWithResult in download_alert.dart.
  • Adjusted background color assignment in download_alert.dart for better code clarity.

Other Changes:

  • Changed the return type of build method in BookDetailsNotifier to FutureOr<CategoryFeed>.
  • Refactored the constructor of BookDetailsNotifierProvider for improved initialization.

… and compatibility

- Updated various package versions in `pubspec.yaml` and `pubspec.lock`, including `auto_route`, `permission_handler`, `share_plus`, and others.
- Refactored notifier classes for better structure and readability.
- Adjusted UI components to align with the latest Flutter material theme changes.
- Minor code cleanups and optimizations across multiple files, including database configuration and router setup.
- Ensured compatibility with Flutter SDK version updates.
- Removed generated files for various notifiers and router classes to reduce clutter and improve maintainability.
- Updated .gitignore to exclude additional generated Dart files and the pubspec.lock file.
- Ensured that the project structure aligns with best practices for generated code management.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant