Skip to content

Weekly Permissions sync 2025-08-23#1235

Closed
marabooy wants to merge 2 commits intodevfrom
permissions-update/2025-08-23
Closed

Weekly Permissions sync 2025-08-23#1235
marabooy wants to merge 2 commits intodevfrom
permissions-update/2025-08-23

Conversation

@marabooy
Copy link
Copy Markdown
Contributor

Weekly Permissions sync 2025-08-23

Copilot AI review requested due to automatic review settings August 23, 2025 00:30
@marabooy marabooy requested a review from a team as a code owner August 23, 2025 00:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a weekly permissions sync that introduces a new Teams permission scope and updates file storage endpoint permissions. The changes expose a previously hidden permission and add granular control for team member management.

  • Adds new TeamMember.ReadWriteNonOwnerRole.All permission for managing non-owner team members
  • Adds file storage recycle bin restore endpoint with specific drive item ID targeting
  • Updates provisioning info to make a permission visible (unhidden)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
permissions/new/permissions.json Adds new Teams permission scope and file storage recycle bin endpoint
permissions/new/ProvisioningInfo.json Updates permission visibility by changing isHidden from true to false
Comments suppressed due to low confidence (2)

permissions/new/permissions.json:11963

  • The path uses the same parameter name '{id}' for both team ID and member ID, which creates ambiguity. Consider using distinct parameter names like '/teams/{teamId}/members/{memberId}' for clarity.
            "Application"

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jasonjoh
Copy link
Copy Markdown
Member

Closing as 0 files changed

@jasonjoh jasonjoh closed this Aug 28, 2025
@jasonjoh jasonjoh deleted the permissions-update/2025-08-23 branch August 28, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants