[gui] add 'Launch & Configure next' button to launch form

[levkropp]

resolves #4028

This pull request introduces a new feature to the LaunchForm in the src/client/gui/lib/catalogue/launch_form.dart file, allowing users to launch and configure the next item without closing the form. The most important changes include adding a new button, modifying the launch method to support the new functionality, and adjusting the form reset behavior.

New feature for launching and configuring the next item:

• Added a launchAndConfigureNextButton to the form, which allows users to launch the current item and immediately configure the next one. [1] [2]

Modifications to the launch method:

• Updated the launch method to accept an optional configureNext parameter, which controls whether the form should reset after launching.

Form reset behavior:

• Adjusted the form reset logic to only reset the form and close the drawer if configureNext is false. [1] [2]

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.56%. Comparing base (f064787) to head (da46703).
⚠️ Report is 3427 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4029   +/-   ##
=======================================
  Coverage   89.56%   89.56%           
=======================================
  Files         261      261           
  Lines       14767    14767           
=======================================
  Hits        13226    13226           
  Misses       1541     1541           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[andrei-toterman]

Do you really need to clear those? Won't they just go away with the Scaffold.of(context).closeEndDrawer();?

[andrei-toterman]

Yup, LGTM!

[andrei-toterman]

Nope, does not LGTM! I approved a bit too early apparently 😅 I found a bug.
Sometimes I forget that Dart is not like C++ and every object is passed by reference all the time. And since we're reusing the launch request and mount list objects, reassigning their values messes with places that already use them. So we need to copy them before passing them to the launching code. You'll need to replace this

initiateLaunchFlow(ref, launchRequest, mountRequests);

with this

initiateLaunchFlow(
  ref,
  launchRequest.deepCopy(),
  mountRequests.map((r) => r.deepCopy()).toList(),
);

[Sploder12]

Code looks good but it doesn't build for me or CI

[Sploder12]

LGTM!

[Sploder12]

LGTM!