Skip to content
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

Update using-admin-roles.md #8194

Open
wants to merge 3 commits into
base: public
Choose a base branch
from

Conversation

RuiPereiraTabares
Copy link
Contributor

repro steps
give teams comunications administrator role
that admin witht this role wont be able to set-csuser -enteprisevoicenabled or even assign a lineuri
the set-csuser wont be loaded for him

however
giving the teams administrator rol will work,
they are some dcrs opened by this, changes from the rols are planned but high level customers asking to have this on public documentation.

@ndambrosio
Copy link
Member

Skype for Business admin role would also give them access to the run the cmdlet without all of the additional privileges a Teams Administrator would have.

@scanum
Copy link

scanum commented Nov 30, 2021

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

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

Copt review - @RuiPereiraTabares, kindly commit the suggested changes if you agree with them. Many thanks.

accepted

Co-authored-by: JohanFreelancer9 <[email protected]>
@RuiPereiraTabares
Copy link
Contributor Author

@JohanFreelancer9 Johan my bad i have just accepted the commint

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

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

Apologies @RuiPereiraTabares, one more, please.

Co-authored-by: JohanFreelancer9 <[email protected]>
@RuiPereiraTabares
Copy link
Contributor Author

done @JohanFreelancer9

@scanum
Copy link

scanum commented Mar 22, 2022

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@scanum
Copy link

scanum commented Dec 12, 2022

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@scanum
Copy link

scanum commented Oct 2, 2023

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

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

Successfully merging this pull request may close these issues.

6 participants