💡 [Feature]: Add 'check output window to follow the progress' message to every CLI Action

[Adam-it]

🎯 Aim of the feature

In deploy action we show the following message

The 'check [output window]...' part is really helpful as it is easy to transfer to the VS Code extension output and check what it is currently doing and what CLI for Microsoft 365 commands are done under the hood to make this happen

Unfortunately this was only added to this single action.
We should:

1. recheck the https://github.com/pnp/vscode-viva/blob/main/src/services/CliActions.ts and in every place where we have window.withProgress... add the Check [output window](command:${Commands.showOutputChannel}) to follow the progress. sentence to the loader/progress message

📷 Images (if possible) with expected result

No response

🤔 Additional remarks or comments

No response

[DevPio]

shoot it over to me 😎.

[Adam-it]

shoot it over to me 😎.

Rockstar 🤩

[Adam-it]

@DevPio I just added and hacktoberfest label to this issue. If you open a PR for this issue over October it will count towards the hacktoberfest event 🤩

[DevPio]

Do you want the functionality to ensure that all CLI command executions automatically display the status in the VS Code output window, along with adding Notifications.info for tracking changes on screen and Notifications.error in case of an error?

[Adam-it]

Do you want the functionality to ensure that all CLI command executions automatically display the status in the VS Code output window, along with adding Notifications.info for tracking changes on screen and Notifications.error in case of an error?

it's not about every CLI command execution but every that is done over the CLI actions.
when we use the window.withProgress we should also include the way to easily transfer to the output window.
Currently we for example have something like

    await window.withProgress({
      location: ProgressLocation.Notification,
      title: 'Granting API permissions for the current project...',
      cancellable: true
    }, async (progress: Progress<{ message?: string; increment?: number }>) => {
    .....

the issue is about changing this to

     await window.withProgress({
      location: ProgressLocation.Notification,
      title: 'Granting API permissions for the current project. Check [output window](command:${Commands.showOutputChannel}) to follow the progress',
      cancellable: true
    }, async (progress: Progress<{ message?: string; increment?: number }>) => {
    .....

The same way as we already have for the deploy action.
That way users in the notify will get the link that will open the output window to show the CLI logs.
and it's only that. nothing more 😉

[Adam-it]

@DevPio how's it going? Could you share some update on this issue? Do you need any help from my side to get you unblocked?
Or if anything changed please do let me know so that we may open it up from someone else which is also perfectly fine 👍.

The CLIActions got a bit more code and functionality so just to clarify we should add the output link in the progress notify only for the following actions

not to every action which is out there in this class

[Adam-it]

@DevPio any update on this issue? Do you need any additional help from my side to get you unblocked?
If something changed please and you may not commit to this issue please do let me know so that we may open it up for other 👍

[Adam-it]

@DevPio ping 👆🙏

[DevPio]

Hi Adam, feel free to leave the issue open and move on to another one. Unfortunately, I’ve been too busy to work on it. My apologies.

[Adam-it]

Hi Adam, feel free to leave the issue open and move on to another one. Unfortunately, I’ve been too busy to work on it. My apologies.

no problem. Thanks for the response

[nicodecleyre]

Can I work on this one?

[Adam-it]

Can I work on this one?

Sure

[Adam-it]

And another one! This will also be part of the next pre-release.
Amazing work 👏👏👏