Skip to content

Weekly Permissions sync 2026-03-12#1467

Closed
marabooy wants to merge 4 commits intomasterfrom
permissions-update/2026-03-12
Closed

Weekly Permissions sync 2026-03-12#1467
marabooy wants to merge 4 commits intomasterfrom
permissions-update/2026-03-12

Conversation

@marabooy
Copy link
Contributor

Weekly Permissions sync 2026-03-12

@marabooy marabooy requested a review from a team as a code owner March 12, 2026 00:49
Copilot AI review requested due to automatic review settings March 12, 2026 00:49
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 adding new permission entries and fixing casing in API paths.

Changes:

  • Added new hidden permissions in provisioningInfo.json: DeviceMgmtAgentRecommendations, EntraGalleryIntegrationRequest.*, Privileged*Schedule.*.EntraAppRole, and RoleManagementPolicy.*.EntraAppRole
  • Added a new path /reports/getAppManagementAuditSummary to permissions.json
  • Fixed camelCase casing of privilegedAccess/group API paths (e.g., filterbycurrentuserfilterByCurrentUser, assignmentschedulesassignmentSchedules)

Reviewed changes

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

File Description
permissions/new/provisioningInfo.json Added new hidden permission entries for DeviceMgmtAgentRecommendations, EntraGalleryIntegrationRequest, PrivilegedSchedule EntraAppRole, and RoleManagementPolicy EntraAppRole
permissions/new/permissions.json Added new API path, fixed camelCase casing on privilegedAccess group paths

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

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +41310 to +41312
"/identityGovernance/privilegedAccess/group/assignmentscheduleInstances": "",
"/identityGovernance/privilegedAccess/group/assignmentscheduleInstances/{id}": "",
"/identityGovernance/privilegedAccess/group/assignmentscheduleInstances/filterByCurrentUser(on={value})": "",
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

Inconsistent casing: these three paths use assignmentscheduleInstances (lowercase "schedule", uppercase "Instances") while the other paths in this same block use assignmentScheduleRequests and assignmentSchedules (uppercase "S" in "Schedule"). This looks like an incomplete casing fix — it should be assignmentScheduleInstances to match the camelCase convention used in the corrected paths above (lines 41266-41267).

Copilot uses AI. Check for mistakes.
@jasonjoh jasonjoh closed this Mar 13, 2026
@jasonjoh jasonjoh deleted the permissions-update/2026-03-12 branch March 13, 2026 20:05
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