Skip to content

Weekly Permissions sync 2026-08-01 - #1590

Merged
Jason Johnston (jasonjoh) merged 1 commit into
masterfrom
permissions-update/2026-08-01
Aug 3, 2026
Merged

Weekly Permissions sync 2026-08-01#1590
Jason Johnston (jasonjoh) merged 1 commit into
masterfrom
permissions-update/2026-08-01

Conversation

@marabooy

Copy link
Copy Markdown
Contributor

Weekly Permissions sync 2026-08-01

@marabooy
David (marabooy) requested a review from a team as a code owner August 1, 2026 00:55
Copilot AI review requested due to automatic review settings August 1, 2026 00:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Weekly permissions sync updating the permission catalog and provisioning metadata used by the Graph DevX content pipeline.

Changes:

  • Updated multiple permission definitions (e.g., privilegeLevel adjustments, ownerSecurityGroup updates) and expanded several path mappings (e.g., calls raiseHand/lowerHand).
  • Added new permission entries (e.g., Calls.ReportSyntheticMedia.All, CaseManagement.Read.All, CaseManagement.ReadWrite.All).
  • Updated provisioning deployment metadata (new keys added; visibility/enabled state updates for some permissions).

Reviewed changes

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

File Description
permissions/new/provisioningInfo.json Updates provisioning deployment entries, including new permission keys and metadata changes.
permissions/new/permissions.json Updates the permission catalog (definitions, privilege levels, ownership, and path mappings) and adds new permissions.
Suppressed comments (1)

permissions/new/permissions.json:9487

  • This change updates paths under the permission key "Calls.JoinGroupCallasGuest.All" (lowercase 'a' in 'as'), but the repo consistently uses "Calls.JoinGroupCallAsGuest.All" (including in provisioningInfo.json and permissions-descriptions*.json). Keeping both keys risks documenting/processing the wrong permission and leaves the typo entry with empty adminDisplayName/adminDescription and no privilegeLevel.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread permissions/new/provisioningInfo.json
@jasonjoh
Jason Johnston (jasonjoh) merged commit d60c9c7 into master Aug 3, 2026
5 checks passed
@jasonjoh
Jason Johnston (jasonjoh) deleted the permissions-update/2026-08-01 branch August 3, 2026 14:00
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