-
Notifications
You must be signed in to change notification settings - Fork 37
Open
Labels
Description
Feature request:
On the course the current code is headed, it can get really messy to manage state and other code.
It'd be best if architecture is implemented when the code hasn't grown too much yet.
I'd like to propose using the Provider package with ChangeNotifier as this is beginner-friendly and least complex to implement.
The Tasks of this issue would be
- Adding provider package and creating a
MultiProvider
usingChangeNotifier
for any state. - Create a ChangeNotifier for handling Authentication and move auth logic into that.
Since we'll be having multiple services as this app demands so, we will use getIt package for easy singleton access across the app.
- Refactor any service class to use GetIt.
This issue can of course be split up as it'll be a lot of code refactoring to be done by two people