Use WidgetRef in singleton to avoid amount of changes

[LeGoffMael]

Hello,

First of all, thank you very much Rémi for your incredible work and support to the flutter community.

I recently finished migrating my flutter application from riverpod 0.13.11 to the latest version. Even though it is now done, I am a little annoyed by the amount of changes this migration required due to the new references to the ref parameter.

I would really like to understand where I could reduce its use to only provide it when I really need it.

For example, I am using riverpod in both router and services to secure authenticates routes or logout user.

so instead of providing ref to my custom router as such

MyRouter.smartNavigateNamed(context, ref, redirection, arguments: arguments);

I thought it would be a good idea to simply change the MyRouter class to a singleton, and provide WidgetRef at init.

/// in [main.dart]
@override
void initState() {
  MyRouter.initialize(ref);
  ...
}


/// in [MyRouter.dart] class
class MyRouter {
  static MyRouter? _instance;

  MyRouter._();

  static MyRouter get getInstance => _instance ??= MyRouter._();

  static late WidgetRef _ref;

  static void initialize(WidgetRef ref) {
    _ref = ref;
  }
}


/// other [file.dart]
MyRouter.smartNavigateNamed(context, redirection, arguments: arguments);

in this case, i won't need to change all my widgets to ConsumerWidget or ConsumerStatefulWidget when they only include a call to MyRouter for example.

I tested it manually a bit and it worked fine.

But since I couldn't find any mention of this usage, is this a bad practice? If so, do I have no choice but to update all my routers and service calls 😢 ?

[LeGoffMael]

Is it that bad practice ? 😅