Conversation
There was a problem hiding this comment.
Pull request overview
Weekly permissions sync updating the permissions catalog to expose and describe the Group-XTenantIdentitySync.Read.All permission.
Changes:
- Unhide
Group-XTenantIdentitySync.Read.AllinprovisioningInfo.jsonfor both DelegatedWork and Application schemes. - Add the full permission definition for
Group-XTenantIdentitySync.Read.All(display names/descriptions, pathSets, owner) inpermissions.json.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| permissions/new/provisioningInfo.json | Flips isHidden from true to false for the permission’s delegated + application IDs. |
| permissions/new/permissions.json | Introduces the permission’s descriptive metadata, path coverage, and owner group. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "adminDisplayName": "Read all Group Cross-Tenant Identity Synchronization properties", | ||
| "adminDescription": "Allows the app to read all Cross-Tenant Identity Synchronization properties on Groups, on behalf of the signed-in user.", | ||
| "userDisplayName": "Read all Group Cross-Tenant Identity Synchronization properties", | ||
| "userDescription": "Allows the app to read all Cross-Tenant Identity Synchronization properties on Groups, on behalf of the signed-in user.", | ||
| "requiresAdminConsent": true, |
There was a problem hiding this comment.
New permission strings use inconsistent capitalization compared to the rest of permissions.json (e.g., "Read all Group…" / "on Groups"). Elsewhere these strings consistently use sentence case ("group"/"groups"). Consider updating the displayName/description fields here to match the established casing so UI text stays consistent.
| "adminDisplayName": "Read all Group Cross-Tenant Identity Synchronization properties", | ||
| "adminDescription": "Allows the app to read all Cross-Tenant Identity Synchronization properties on Groups, without a signed-in user.", | ||
| "requiresAdminConsent": true, |
There was a problem hiding this comment.
Same capitalization/style issue as above for the Application scheme strings ("Read all Group…" / "on Groups"). For consistency with other entries, prefer sentence case ("group"/"groups") across display names and descriptions.
2 participants
Weekly Permissions sync 2026-02-10