Skip to content

Update site-post-permissions.md#9823

Open
mmascolino wants to merge 1 commit intomicrosoftgraph:mainfrom
mmascolino:patch-2
Open

Update site-post-permissions.md#9823
mmascolino wants to merge 1 commit intomicrosoftgraph:mainfrom
mmascolino:patch-2

Conversation

@mmascolino
Copy link
Contributor

the permissions table for this graph api item was not correct/complete.

Important

Required for API changes:

  • [] Link to API.md file: ADD LINK HERE
  • [] Link to PR for public-facing schema changes (schema-Prod-beta/v1.0.csdl): ADD LINK HERE
  • [] OPTIONAL: Attach the documentation plan generated for AI-assisted docs authoring and validation to help speed up content review.

Add other supporting information, such as a description of the PR changes:

ADD INFORMATION HERE


Important

The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.

After you've created your PR, expand this section for tips and additional instructions.
  • do not merge is the default PR status and is automatically added to all open PRs that don't have the ready to merge label.
  • Add the ready for content review label to start a review. Only PRs that have met the minimum requirements for content review and have this label are reviewed.
  • If your content reviewer requests changes, review the feedback and address accordingly as soon as possible to keep your pull request moving forward. After you address the feedback, remove the changes requested label, add the review feedback addressed label, and select the Re-request review icon next to the content reviewer's alias. If you can't add labels, add a comment with #feedback-addressed to the pull request.
  • After the content review is complete, your reviewer will add the content review complete label. When the updates in this PR are ready for external customers to use, replace the do not merge label with ready to merge and the PR will be merged within 24 working hours.
  • Pull requests that are inactive for more than 6 weeks will be automatically closed. Before that, you receive reminders at 2 weeks, 4 weeks, and 6 weeks. If you still need the PR, you can reopen or recreate the request.

For more information, see the Content review process summary.

the permissions table for this graph api item was not correct/complete.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 9c60e33:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/site-post-permissions.md ✅Succeeded

For more details, please refer to the build report.

@Danipocket
Copy link
Contributor

@mmascolino, are you a MS FTE? If so, please use the private repo and coordinate with the permissions team.

@mmascolino
Copy link
Contributor Author

@Danipocket - i am not. just a concerned citizen.

@Danipocket
Copy link
Contributor

@Danipocket - i am not. just a concerned citizen.

:P Let me keep this PR open and find someone from this workload to validate this change. I’m sure you’re correct. Thanks again! @mmascolino

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.

2 participants