Adam Wรณjcik
Adam Wรณjcik
merged manually. Thank you for your amazing work ๐ and sorry for the delay ๐. you rock ๐คฉ
I may take this one ๐ there is still a lot of those we need to have and I guess I should spend some time and create issue for each...
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 comments here ๐ค? I suppose very similar to the one on the list level which I already implemented and is merges so this one should be quite...
> Updated the specs ๐ thanks @martinlingstuyl for updating the spec. I thought it's on me ๐. Lets open it up for others. I will try to catch up with...
ok I got my current work done. I may take this one
@martinlingstuyl do you think you could do an additional review on this one ๐ from me it's ready to be merged (with a small fix along the way I will...
> Hi @Adam-it, I can review the code, but I do not have a pc with me (on holiday) so I can't try it out. I assume that's okay, as...
> Hi @michaelmaillot, @Adam-it, I've left some comments and considerations.๐ great ๐คฉ Thanks @martinlingstuyl for the double check ๐๐๐. I didn't feel confident enough in merging this ๐ and I...
@michaelmaillot thanks for you amazing work ๐คฉ. I will try to review it asap and align to our latest approach with commands which was refactored ๐