webfactorymk / flutter-template

Flutter template project - Simple ToDo app with scalable project structure.
https://webfactory.mk/
MIT License
166 stars 36 forks source link

Fix DataNotificationManager documentation #49

Closed petrovWF closed 1 year ago

petrovWF commented 2 years ago

This documentations contains false information(and it can be misleading) for example: /// To obtain an instance use serviceLocator.get<NotificationsManager>()

This is not correct and should be /// To obtain an instance use serviceLocator.get<NotificationsConsumer>()

Also we should register DataNotificationManager instance with the serviceLocator..

If we implement this changes please consider changing the logs inside DataNotificationManager that contains: NotificationManager into NotificationConsumer..