Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update-M365DSCResourceDocumentationPage cmdlet not working #2930

Closed
ricmestre opened this issue Feb 23, 2023 · 2 comments · Fixed by #2931 or #2953
Closed

Update-M365DSCResourceDocumentationPage cmdlet not working #2930

ricmestre opened this issue Feb 23, 2023 · 2 comments · Fixed by #2931 or #2953
Labels
Bug Something isn't working Core Engine

Comments

@ricmestre
Copy link
Contributor

The latest update to 1.23.222.1 broke Update-M365DSCResourceDocumentationPage cmdlet, this is due to settings.json for resource IntuneDeviceEnrollmentConfigurationWindows10 since it contains two trailing commas that shouldn't be there and it errors out on ConvertFrom-Json.

{
    "resourceName": "IntuneDeviceEnrollmentConfigurationWindows10",
    "description": "This resource configures an Intune Device Enrollment Status Page Configuration for Windows10.",
    "permissions": {
            },
            "application": {
                "read": [
                    {
                        "name": "DeviceManagementConfiguration.Read.All"
                    },
                    {
                        "name": "DeviceManagementServiceConfig.Read.All"
                    },
                ],
                "update": [
                    {
                        "name": "DeviceManagementConfiguration.ReadWrite.All"
                    },
                    {
                        "name": "DeviceManagementServiceConfig.ReadWrite.All"
                    }
                ]
            }
        }
    }
}
@andikrueger
Copy link
Collaborator

This issue does impact Update-M365DSCAllowedGraphScopes as well:

image

We really need a check to get rid of these issues: #2912

@ykuijs
Copy link
Member

ykuijs commented Feb 23, 2023

Agree with that. I noticed this last evening as well and am already working on adding a QA check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Core Engine
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants