Adam Wรณjcik
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.