Skip to content

Weekly Permissions sync 2026-02-25#1452

Open
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-02-25
Open

Weekly Permissions sync 2026-02-25#1452
marabooy wants to merge 1 commit intomasterfrom
permissions-update/2026-02-25

Conversation

@marabooy
Copy link
Contributor

Weekly Permissions sync 2026-02-25

Copilot AI review requested due to automatic review settings February 25, 2026 00:46
@marabooy marabooy requested a review from a team as a code owner February 25, 2026 00:46
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

Weekly permissions sync updating provisioningInfo.json to reflect the latest permission deployment metadata (notably filling in missing GUIDs and introducing new hidden permissions).

Changes:

  • Populated previously empty id fields for several AppManagementOnboarding* permission deployments.
  • Added new (hidden) permission deployment entries for AppManagementOnboarding*Test*, EngagementExport.Read.All, and SecurityIdentitiesMigration.*.

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

Comment on lines +13448 to +13453
{
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The new provisioning entry for SecurityIdentitiesMigration.ReadWrite.All is missing the id field for the Application scheme. Please add the appropriate GUID (or an explicit empty string if intentionally unknown) to match the schema used by the surrounding SecurityIdentities* permissions.

Copilot uses AI. Check for mistakes.
Comment on lines +13425 to +13430
{
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The new provisioning entry for SecurityIdentitiesMigration.Read.All is missing the id field for the DelegatedWork scheme. Other entries in this file consistently include an id (sometimes empty) as the first property; please add the correct GUID (or an explicit empty string if intentionally unknown) to match the expected schema.

Copilot uses AI. Check for mistakes.
Comment on lines +13432 to +13437
{
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The new provisioning entry for SecurityIdentitiesMigration.Read.All is missing the id field for the Application scheme. Please add the appropriate GUID (or an explicit empty string if intentionally unknown) to keep the object shape consistent with the rest of provisioningInfo.json.

Copilot uses AI. Check for mistakes.
Comment on lines +13441 to +13446
{
"scheme": "DelegatedWork",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "8ee8fdad-f234-4243-8f3b-15c294843740"
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The new provisioning entry for SecurityIdentitiesMigration.ReadWrite.All is missing the id field for the DelegatedWork scheme. Please add the correct GUID (or an explicit empty string if intentionally unknown) so downstream tooling that expects an id property doesn’t break.

Copilot uses AI. Check for mistakes.
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.

2 participants