cli-microsoft365 icon indicating copy to clipboard operation
cli-microsoft365 copied to clipboard

New command: spo list defaultvalue get. Closes #6535

Open Ateina opened this issue 3 months ago • 3 comments

Closes #6535

Ateina avatar Oct 02 '25 20:10 Ateina

Thank you for opening the PR 👏 We will review it ASAP

Adam-it avatar Oct 02 '25 20:10 Adam-it

@Adam-it Thank you for the review 💚 I thought about moving these methods but wasn't sure if it should be done in this PR Will fix all comments and update the PR asap 👍

Ateina avatar Nov 04 '25 21:11 Ateina

@Ateina I saw you added a commit with a fixup I guess. If you think the PR is read for another go and to be merged use the ready for review button to let us know if we may proceed 👍

Adam-it avatar Dec 10 '25 20:12 Adam-it

@Ateina I just noticed your reply to one of my comments when doing the review. So sorry I did not respond sooner. I left some guidance on what would be the best way to proceed. Let me know if this is something you could tweak as well and besides that, we should be ready to merge 👍

Adam-it avatar Dec 14 '25 22:12 Adam-it

@Adam-it Thank you 💚 I’ve moved the duplicated code to the utils. If anything else is needed, no problem to update 🙂

Ateina avatar Dec 18 '25 21:12 Ateina