Allow Wizard.of(context).next() for the last route as well?

[s0nerik]

I have the following wizard setup, which looks quite natural to me and doesn't require the ConfirmAuthPage to do anything with GoRouter (thus, it can be safely moved around without any fear of breaking the flow):

  @override
  Widget build(BuildContext context) {
    return Wizard(
      routes: {
        '/request':
            WizardRoute(builder: (_) => const RequestAuthPage()),
        '/confirm': WizardRoute(
          builder: (_) => const ConfirmAuthPage(),
          onNext: (_) {
            GoRouter.of(context).pop();
            return null;
          },
        ),
      },
    );
  }

The problem is that Wizard doesn't let me call next() from within the ConfirmAuthPage, which seems to leave me no choice other than introducing GoRouter routing right into the ConfirmAuthPage, which I'd want to avoid.

What do you think about adding an option to allow next() on the last page of the flow or even allowing it unconditionally (doing nothing if there's no next page in the flow)?

[andyhorn]

I would also like to have onNext not throw an error on the last page or add an onComplete callback 👍

[spydon]

An onConplete callback sounds like a good idea, do you want to put up a PR for this @andyhorn?