Adam Wรณjcik

Results 1119 comments of Adam Wรณjcik

Thanks @MathijsVerbeeck for all of your awesome work ๐Ÿ‘. I assigned you to the issue. Can't wait to see the PR ๐Ÿ˜. You rock ๐Ÿคฉ

@pnp/cli-for-microsoft-365-maintainers I think this spec is quite straight forward ? I think we may open it up if you agree and I could take it.

> Looks great, but could you change the word subsite into web or site? "Yes we can". But won't it be misleading for the end user as actually you may...

Ok @martinlingstuyl then if you have bo other comments then I will take this one for today

setting this to draft as I want to align this to the last changes done for commands

@martinlingstuyl I agree. Since we already have the `m365 teams message list` with `--channelId` option I would leave it and include a new option which is `--userId`. So I totally...

@martinlingstuyl weren't we suppose to drop the `userName` option for now? Besides this single comment all other seem ok and I think we may open this up. do you think...

good research @martinlingstuyl ๐Ÿ‘. I think the `--includeAttachments` option would be ok and not exporting them by default.

putting this on hold before we agree on some on the approach here https://github.com/pnp/cli-microsoft365/issues/3540

@pnp/cli-for-microsoft-365-maintainers any additional comment here ? if it is ok I may take this one.