-
Notifications
You must be signed in to change notification settings - Fork 307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extend spo listitem
with listUrl
(part 1)
#3889
Conversation
Splendid @MathijsVerbeeck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Just one small detail we may improve along the way and I think we are good to go
@pnp/cli-for-microsoft-365-maintainers anyone up for a second opinion? |
Hi @Adam-it I will resolve the remark tomorrow 😄 |
TBH I noticed those were deprecated but I just couldn't stop myself from refactoring it 😅. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pulled and tested locally, looks good ✔
awesome work @MathijsVerbeeck 👍
you rock 🤩
ready to merge 🚀 |
merged manually. |
Closes #3813