Skip to content

Adds "aad administrativeunit member add" command. Closes #5615 - #5651

Closed
MartinM85 wants to merge 4 commits into
pnp:mainfrom
MartinM85:feature/5615-add-administrative-unit-member
Closed

MartinM85 wants to merge 4 commits into
pnp:mainfrom
MartinM85:feature/5615-add-administrative-unit-member

Conversation

@MartinM85

Copy link
Copy Markdown
Contributor

Closes #5615

@milanholemans

Copy link
Copy Markdown
Contributor

Thank you @MartinM85, we'll try to review it ASAP!

@martinlingstuyl martinlingstuyl self-assigned this Dec 10, 2023
@MartinM85 MartinM85 changed the title Adds "aad administrativeunit member add" command Dec 11, 2023

@martinlingstuyl martinlingstuyl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine @MartinM85, just a couple of remarks. I'll merge it as soon as you've found the time to fix them.

Comment thread src/m365/aad/commands.ts
Comment thread src/utils/aadDevice.spec.ts Outdated
Comment thread src/utils/aadDevice.spec.ts Outdated
Comment thread src/m365/aad/commands/administrativeunit/administrativeunit-member-add.ts Outdated
@martinlingstuyl
martinlingstuyl marked this pull request as draft December 11, 2023 20:22
@MartinM85
MartinM85 marked this pull request as ready for review December 12, 2023 07:42

@martinlingstuyl martinlingstuyl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine like this. I'll merge it soon!

@martinlingstuyl

Copy link
Copy Markdown
Contributor

Merged manually, thank you! 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants