Adam Wójcik
ready to merge 🚀. Before I merge @Saurabh7019 lets check one comment/question I left in the review
Merged manually. Thank you for your awesome work 👍 You Rock 🤩
> @Adam-it, We have a drive.ts utility class, and we should use that instead of driveUtil.ts. I apologize for the delay in addressing this, but I believe this command can...
@arenk good tip. I was not aware of that either 👍. I will research this a bit and lets include it in the docs so it will be clear for...
@pnp/cli-for-microsoft-365-maintainers what @arenk suggested I think is very valid and we could also added to our docs for the SPE commands. I would probably suggest we add something generic like...
Please check other commands that have some small naming corrections + additional site-relative URL for `folderUrl`. besides that LGTM. @pnp/cli-for-microsoft-365-maintainers, anyone else?
@andrewconnell wanna take a look at 👆
> Aren't these quite a lot of default properties @Adam-it? Can we fit that much info on a default-sized terminal screen? good comment. I reduced to fewer props. @pnp/cli-for-microsoft-365-maintainers any...
good find and quick action has been taken already. Your Rock 🤩
Awesome suggestion 👏. Let's get this 🚢