Conversation
There was a problem hiding this comment.
Pull Request Overview
This is a weekly permissions sync that updates API endpoint permissions and provisioning information for Microsoft Graph services. The changes include adding new endpoints for device management and lifecycle workflows, along with modifying existing permissions for organization places.
- Added new API endpoints for device management virtual endpoints and lifecycle workflows
- Updated permissions configuration for organization places to remove Application scheme
- Added new provisioning entries for delegated work permissions
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| permissions/new/permissions.json | Added new API endpoints and updated permission schemes for virtual endpoints, lifecycle workflows, and organization places |
| permissions/new/ProvisioningInfo.json | Added new provisioning entries with delegated work scheme permissions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "/devicemanagement/virtualendpoint/cloudpcs/{id}": "least=DelegatedWork,Application", | ||
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/getcloudpcconnectivityhistory": "least=DelegatedWork,Application", | ||
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/getCloudPcLaunchInfo": "least=DelegatedWork,Application", | ||
| "/devicemanagement/virtualEndpoint/cloudPcs/{id}/retrieveFrontlineCloudPcDetail": "least=DelegatedWork,Application", |
There was a problem hiding this comment.
The URL path uses inconsistent casing. 'virtualEndpoint' and 'cloudPcs' use camelCase while other similar paths in the same section use lowercase (e.g., 'virtualendpoint/cloudpcs'). This should match the existing pattern for consistency.
| "/devicemanagement/virtualEndpoint/cloudPcs/{id}/retrieveFrontlineCloudPcDetail": "least=DelegatedWork,Application", | |
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/retrievefrontlinecloudpcdetail": "least=DelegatedWork,Application", |
| "/devicemanagement/virtualendpoint/cloudpcs/{id}": "", | ||
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/getcloudpcconnectivityhistory": "", | ||
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/getCloudPcLaunchInfo": "", | ||
| "/devicemanagement/virtualEndpoint/cloudPcs/{id}/retrieveFrontlineCloudPcDetail": "", |
There was a problem hiding this comment.
Same casing inconsistency as above. The path should use lowercase 'virtualendpoint/cloudpcs' to match the existing API path conventions in this section.
| "/devicemanagement/virtualEndpoint/cloudPcs/{id}/retrieveFrontlineCloudPcDetail": "", | |
| "/devicemanagement/virtualendpoint/cloudpcs/{id}/retrievefrontlinecloudpcdetail": "", |
| @@ -12558,6 +12566,14 @@ | |||
| "isHidden": true, | |||
| "isEnabled": true, | |||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |||
There was a problem hiding this comment.
The resourceAppId is missing the final '0' character. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource app ID format used elsewhere in the file.
Weekly Permissions sync 2025-09-19