-
Notifications
You must be signed in to change notification settings - Fork 370
New command: viva engage community remove. Closes #6280 #6286
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
New command: viva engage community remove. Closes #6280 #6286
Conversation
|
Thank you, we'll try to review it ASAP! |
16541f8 to
da4a4e9
Compare
da4a4e9 to
75602cd
Compare
d9b2137 to
7e5ca93
Compare
7e5ca93 to
324175e
Compare
|
Hi @MathijsVerbeeck, will you have time to check it? It's the last missing command from the |
MathijsVerbeeck
left a comment
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.
Hi @MartinM85
Sorry for the very long delay. Some very minor remarks. Could you please look at them?
324175e to
1f62a22
Compare
MathijsVerbeeck
left a comment
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.
Good job Martin. I will merge this one as soon as possible.
|
@pnp/cli-for-microsoft-365-maintainers Any plan to merge this? |
|
I'll take it |
Adam-it
left a comment
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.
@MartinM85 Awesome work 👏👏👏👏
|
Ready to merge 🚀 |
|
Merged manually 👍 |

Closes #6280