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.

Sign up for GitHub

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

Jump to bottom

Update notes for Directory.ReadWrite.All #9029

Merged
merged 3 commits into from
Jan 24, 2024
Merged

Update notes for Directory.ReadWrite.All #9029

merged 3 commits into from
Jan 24, 2024

Conversation

Jackson-Woods
Copy link
Member

  • update summary statement to emphasize that Dir.RW.All grants rights to grant rights
  • removed old verbage about Directory permissions governing basic resources that now have specific permissions

@Jackson-Woods
Update notes for Directory.ReadWrite.All
01181eb 
- update summary statement to emphasize that Dir.RW.All grants rights to grant rights
- removed old verbage about Directory permissions governing basic resources that now have specific permissions
@microsoft-github-policy-service microsoft-github-policy-service bot added the do not merge Use this label to indicate to the repository admins that your PR Is not ready to merge into master. label Jan 19, 2024
@learn-build-service-prod Learn Build Service (PROD)
Copy link

Learn Build status updates of commit 01181eb:

✅ Validation status: passed

File Status Preview URL Details
includes/permissions-notes/directory.readwrite.all.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jackson-Woods
Revised note
8813692 
Change to a blanket warning about the permission
@learn-build-service-prod Learn Build Service (PROD)
Copy link

Learn Build status updates of commit 8813692:

✅ Validation status: passed

File Status Preview URL Details
includes/permissions-notes/directory.readwrite.all.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

FaithOmbongi
FaithOmbongi approved these changes Jan 22, 2024
View reviewed changes
Copy link
Collaborator

@FaithOmbongi FaithOmbongi left a comment

Choose a reason for hiding this comment

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

LGTM!

@learn-build-service-prod Learn Build Service (PROD)
Copy link

Learn Build status updates of commit 02e6445:

✅ Validation status: passed

File Status Preview URL Details
includes/permissions-notes/directory.readwrite.all.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@FaithOmbongi FaithOmbongi added area: Entra APIs and issues relating to Microsoft Entra (previously Azure AD) features. blocked PRs that are blocked from content review or getting merged for some reason. labels Jan 22, 2024
@FaithOmbongi FaithOmbongi self-assigned this Jan 22, 2024
@FaithOmbongi FaithOmbongi added content review complete Apply when the content review is complete on a PR. ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master. and removed awaiting internal response blocked PRs that are blocked from content review or getting merged for some reason. do not merge Use this label to indicate to the repository admins that your PR Is not ready to merge into master. labels Jan 23, 2024
@jasonjoh jasonjoh added this to the Community Contributions milestone Jan 23, 2024
@Lauragra Lauragra merged commit 6e145d9 into microsoftgraph:main Jan 24, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FaithOmbongi FaithOmbongi FaithOmbongi approved these changes

Assignees

@FaithOmbongi FaithOmbongi

Labels
area: Entra APIs and issues relating to Microsoft Entra (previously Azure AD) features. content review complete Apply when the content review is complete on a PR. ready to merge Use this label to indicate to the repository admins that your PR is ready to merge into master.
Projects
None yet
Milestone
Community Contributions
Development

Successfully merging this pull request may close these issues.
4 participants
@Jackson-Woods @FaithOmbongi @Lauragra @jasonjoh