Adam Wójcik comments

Results 1119 comments of


                                            Adam Wójcik

Align options with naming convention

merged to v9

New command 'entra enterpriseapp permission remove' - Remove an app-only or delegated permissions from an enterprise application

LGTM. The new command brings a lot more clarity than the once we already have 👍. Some small comments from my side: - we are missing the `force` option since...

New command: `viva engage community add`

> Yes indeed, as discussed before, we'll just leave the group commands untouched and just introduce 2 new `community` commands. Thats a bit of a bummer but in the end...

Fixes 'spo file add' check-in issue

ready to merge 🚀

Fixes 'spo file add' check-in issue

Merged manually 👍 Awesome work 👏👏 You Rock 🤩

`spo file add`: A file or folder with the name already exists

Thanks for rising the issue. We will look into it ASAP

New command: m365 spe containertype get

@andrewconnell any feed on 👆

New command: m365 spe containertype get

> The only remark from my side is that `get` commands don't implement default properties. Thanks for the comment. Removed. @pnp/cli-for-microsoft-365-maintainers any other comment before I open this up?

New command: `m365 spo tenant site membership list`

hi all, sorry for coming late to the party 🙏. - @mkm17 isn't this more like a `list` command rather than a `get` command? As we are more like listing...

New command: `m365 spo tenant site membership list`

yes lets open it up 💪 thanks, @pnp/cli-for-microsoft-365-maintainers for your double check on this issue. @mkm17 wanna take the lead on this or leave it for someone else?