thewckd / blocarchitecturestartupproject Goto Github PK

As the app grows in size, the routes will also grow as well and keeping the text Strings used in the app and the routes in the same place will be anti-pattern.
I suggest you to create a utils folder and under it keep the routes.dart file which will hold all the routes.

If you are ok with this idea, I can create a PR with the changes

// @OverRide
// void onChange(Cubit cubit, Change change) {
// print(change);
// super.onChange(cubit, change);
// }

// @OverRide
// void onClose(Cubit cubit) {
// super.onClose(cubit);
// }

// @OverRide
// void onCreate(Cubit cubit) {
// print(cubit);
// super.onCreate(cubit);
// }

// @OverRide
// void onError(Cubit cubit, Object error, StackTrace stackTrace) {
// super.onError(cubit, error, stackTrace);
// }

These are removed in latest version of Bloc & Flutter Bloc. I think they don't support Cubit but some or none supports Bloc (In above four override methods of BlocObserver Class.

The architecture is missing test folder & test.dart files. Please add them as they are necessary for the development stage.