sql-docs icon indicating copy to clipboard operation
sql-docs copied to clipboard

Update permission references and add a note about additional required permissions

Open rufer7 opened this issue 9 months ago • 6 comments

This pull request includes changes to the azure-sql/database/authentication-azure-ad-user-assigned-managed-identity.md file to update permission references and add a note about additional required permissions for creating a contained database user for a Microsoft Entra group.

Permission reference updates:

  • Updated the URLs for User.Read.All, GroupMember.Read.All, and Application.Read.All to correct paths.

Additional permissions note:

  • Added a note indicating that the Group.Read.All permission is also required for the creation of a contained database user for a Microsoft Entra group.

rufer7 avatar Feb 27 '25 19:02 rufer7

@rufer7 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Feb 27 '25 19:02 prmerger-automator[bot]

Learn Build status updates of commit e1c2b07:

:white_check_mark: Validation status: passed

File Status Preview URL Details
azure-sql/database/authentication-azure-ad-user-assigned-managed-identity.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@VanMSFT

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Feb 27 '25 21:02 v-dirichards

Learn Build status updates of commit ee91508:

:white_check_mark: Validation status: passed

File Status Preview URL Details
azure-sql/database/authentication-azure-ad-user-assigned-managed-identity.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Thanks, @rufer7 - I'm checking on this.

VanMSFT avatar Feb 27 '25 21:02 VanMSFT

@vanmsft Can you review this old PR and determine whether it needs to be closed or merged?

IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. If the changes no longer apply, add an explanation and a #please-close comment to close the PR.

@MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Jul 10 '25 21:07 v-dirichards