vscode-viva icon indicating copy to clipboard operation
vscode-viva copied to clipboard

🐞 Bug report: Grant API permissions action works but presents error

Open Adam-it opened this issue 1 year ago β€’ 5 comments

⭐ Priority

(Medium)😒 I'm annoyed but I'll live

πŸ“ Describe the bug

the grant API permission action seems to be failing with some cryptic error message image

even though it seems to work properly image

I did some investigation and it seems CLI for Microsoft 365 is failing when running m365 spfx project permissions grant

image

So in order to fix this issue we should first open a bug in CLI for Microsoft 365 and have it fixed there as well. After it will get fixed in that repo we should align our commonjs CLI copy and restest

πŸ‘£ Steps To Reproduce

just run Grant API permissions on any SPFx project

πŸ“œ Expected behavior

should grant permissions, this works, but shouldn't present any error message

πŸ“· Screenshots

added above

❓SharePoint Framework Toolkit version

3.3.0

❓Node.js version

18.18.2

πŸ€” Additional context

No response

Adam-it avatar Jun 30 '24 21:06 Adam-it

Let me look into this issue. Also, I will create a bug report in the CLI repo if one doesn't already exist.

Saurabh7019 avatar Jul 03 '24 08:07 Saurabh7019

Let me look into this issue. Also, I will create a bug report in the CLI repo if one doesn't already exist.

All yours. No the bug on CLI repo still doesn't exist. I didn't have time to add it yet

Adam-it avatar Jul 03 '24 09:07 Adam-it

still doesn't exist

actually, it does. https://github.com/pnp/cli-microsoft365/issues/5889

Saurabh7019 avatar Jul 03 '24 09:07 Saurabh7019

still doesn't exist

actually, it does. https://github.com/pnp/cli-microsoft365/issues/5889

AAA awesome, I didn't notice that one. Sorry it is just to much for me sometimes πŸ˜‰. Now let's resolve it on CLI side and then we may update the CJS version of CLI and it should 'just' work here πŸ˜…

Adam-it avatar Jul 03 '24 09:07 Adam-it

We should wait until the related issue in CLI repo will get fixed

Adam-it avatar Nov 18 '24 21:11 Adam-it