Skip to content

Conversation

@marabooy
Copy link
Contributor

Weekly Permissions sync 2025-09-25

@marabooy marabooy requested a review from a team as a code owner September 25, 2025 00:26
Copilot AI review requested due to automatic review settings September 25, 2025 00:26
Copy link
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 adds new Copilot conversation deletion permissions and expands several API permissions for backup/restore, education, and SharePoint migration functionalities.

  • Adds "CopilotConversation.Delete" permission with delegated work scheme
  • Expands backup/restore permissions to include full backup functionality for drives, mailboxes, and sites
  • Adds comprehensive education assignment management permissions for application-only access
  • Updates SharePoint migration permissions to include cross-organization user and group mappings

Reviewed Changes

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

File Description
permissions/new/provisioningInfo.json Adds CopilotConversation.Delete permission configuration
permissions/new/permissions.json Expands API permissions across backup/restore, education, and SharePoint migration domains
Comments suppressed due to low confidence (1)

permissions/new/permissions.json:1

  • The removal of '/education/classes/{id}/assignments' and '/education/classes/{id}/assignments/{id}/resources' paths from this permission block appears to be restructuring these permissions. However, these same paths are then added back in separate permission blocks below. This creates potential confusion about permission inheritance and could lead to duplicate or conflicting permission definitions.
{

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

@jasonjoh jasonjoh merged commit 02cfd59 into dev Sep 25, 2025
4 checks passed
@jasonjoh jasonjoh deleted the permissions-update/2025-09-25 branch September 25, 2025 14:33
MIchaelMainer added a commit that referenced this pull request Oct 2, 2025
* Weekly Permissions sync 2025-09-10 (#1264)

* Weekly Permissions sync 2025-09-11 (#1265)

* Weekly Permissions sync 2025-09-22 (#1276)

* Update provisioningInfo.json (#1278)

* Weekly Permissions sync 2025-09-23 (#1277)

* Weekly Permissions sync 2025-09-23

* Update ProvisioningInfo.json

---------

Co-authored-by: Jason Johnston <jasonjoh@users.noreply.github.com>

* Update permission description for User SoA permissions (#1281)

* Update permission description for User SoA permissions

* Fix description

---------

Co-authored-by: Darshan Maiya <Darshan.Maiya@microsoft.com>

* Weekly Permissions sync 2025-09-23 (#1279)

* Weekly Permissions sync 2025-09-24 (#1280)

Co-authored-by: Jason Johnston <jasonjoh@users.noreply.github.com>

* Weekly Permissions sync 2025-09-25 (#1282)

* Weekly Permissions sync 2025-09-26 (#1283)

* Weekly Permissions sync 2025-09-28 (#1285)

* Weekly Permissions sync 2025-10-01 (#1288)

* Weekly Permissions sync 2025-10-02 (#1289)

* release: deploy to prod with updated pipeline (#1230) (#1292)

* Weekly Permissions sync 2025-07-31

* Update permission description for Contacts SoA permissions (#1215)



---------

Co-authored-by: Ramses Sanchez-Hernandez <63934382+ramsessanchez@users.noreply.github.com>
Co-authored-by: David <1511024+marabooy@users.noreply.github.com>
Co-authored-by: James Ndegwa Maringa <wandeg@users.noreply.github.com>
Co-authored-by: Darshan Maiya <darshanmaiya@users.noreply.github.com>
Co-authored-by: Darshan Maiya <Darshan.Maiya@microsoft.com>

---------

Co-authored-by: David <1511024+marabooy@users.noreply.github.com>
Co-authored-by: Jason Johnston <jasonjoh@users.noreply.github.com>
Co-authored-by: Darshan Maiya <darshanmaiya@users.noreply.github.com>
Co-authored-by: Darshan Maiya <Darshan.Maiya@microsoft.com>
Co-authored-by: Ramses Sanchez-Hernandez <63934382+ramsessanchez@users.noreply.github.com>
Co-authored-by: James Ndegwa Maringa <wandeg@users.noreply.github.com>
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