Skip to content

Breaking Change - Fix Access Review Definition ScopeValue Type#6857

Draft
FabienTschanz wants to merge 1 commit intomicrosoft:Devfrom
FabienTschanz:fix/access-review-preview
Draft

Breaking Change - Fix Access Review Definition ScopeValue Type#6857
FabienTschanz wants to merge 1 commit intomicrosoft:Devfrom
FabienTschanz:fix/access-review-preview

Conversation

@FabienTschanz
Copy link
Collaborator

Pull Request (PR) description

This PR updates the format for the permission and resource scopes of the AADAccessReviewDefinition resource.
Usually, this could be considered a breaking change because it changes the structure of the object, but there was no possible way to create such an Access Review Definition in the GUI. The only way to create it now is through a feature that's still in preview, called Catalog Reviews - Multi Resource.

image

Because this is only preview for now and was not previously part of the experience, I believe this is justified to be changed. My test policies were not affected by this change and still contained the same definitions as before.

This Pull Request (PR) fixes the following issues

None.

Task list

  • Added an entry to the change log under the Unreleased section of the file CHANGELOG.md.
    Entry should say what was changed and how that affects users (if applicable), and
    reference the issue being resolved (if applicable).
  • Resource parameter descriptions added/updated in the schema.mof.
  • Resource documentation added/updated in README.md.
  • Resource settings.json file contains all required permissions.
  • Examples appropriately added/updated.
  • Unit tests added/updated.
  • New/changed code adheres to DSC Community Style Guidelines.

@FabienTschanz FabienTschanz force-pushed the fix/access-review-preview branch from c2eb27b to a222fc1 Compare January 28, 2026 15:33
@FabienTschanz FabienTschanz changed the title Fix Access Review Definition Scopes Breaking Change - Fix Access Review Definition ScopeValue Type Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants