Skip to content

Weekly Permissions sync 2025-09-06#1258

Merged
jasonjoh merged 1 commit intodevfrom
permissions-update/2025-09-06
Sep 8, 2025
Merged

Weekly Permissions sync 2025-09-06#1258
jasonjoh merged 1 commit intodevfrom
permissions-update/2025-09-06

Conversation

@marabooy
Copy link
Copy Markdown
Contributor

@marabooy marabooy commented Sep 6, 2025

Weekly Permissions sync 2025-09-06

Copilot AI review requested due to automatic review settings September 6, 2025 00:25
@marabooy marabooy requested a review from a team as a code owner September 6, 2025 00:25
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 is a weekly permissions sync that updates permission configurations and adds new API endpoint mappings. The changes include restructuring workbook comment endpoint permissions, adding a new RealTimeActivityFeed permission, and updating Teams app enablement permissions.

  • Reorganizes workbook comments endpoint permissions by moving them to different HTTP method groups
  • Introduces new RealTimeActivityFeed.Read.All permission with Copilot communication endpoints
  • Adds Teams channel enabled apps endpoint permissions across multiple permission scopes

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
permissions/new/permissions.json Updates workbook comment endpoints, adds RealTimeActivityFeed permission with Copilot endpoints, and expands Teams app permissions for channel-level app management
permissions/new/ProvisioningInfo.json Updates environment configurations and enables previously disabled Application scheme permissions
Comments suppressed due to low confidence (2)

permissions/new/permissions.json:1

  • The path '/me/drive/root:/{id}:/workbook/comments/{id}' was changed from '/me/drive/root:/{id}:/workbook/comments' but this modification adds a parameter '{id}' to the endpoint, changing its semantic meaning from accessing all comments to accessing a specific comment. This could be a breaking change that affects API consumers expecting the collection endpoint.
{

permissions/new/permissions.json:1

  • Similar to the previous comment, this path was changed from '/users/{id}/drive/root:/{id}:/workbook/comments' to '/users/{id}/drive/root:/{id}:/workbook/comments/{id}', adding a parameter that changes the endpoint from a collection to a specific resource. This could be a breaking change for API consumers.
{

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

@jasonjoh jasonjoh merged commit 10ba08c into dev Sep 8, 2025
4 checks passed
@jasonjoh jasonjoh deleted the permissions-update/2025-09-06 branch September 8, 2025 18:53
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