-
Notifications
You must be signed in to change notification settings - Fork 485
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
PR: IntuneWindowsAutopilotDeploymentProfile - Initial release #3043
PR: IntuneWindowsAutopilotDeploymentProfile - Initial release #3043
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First review with some of the comments discussed via email.
Haven't checked everything yet.
...ureADHybridJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined.schema.mof
Show resolved
Hide resolved
...65DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/readme.md
Outdated
Show resolved
Hide resolved
...C/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/settings.json
Outdated
Show resolved
Hide resolved
...entProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.schema.mof
Show resolved
Hide resolved
...osoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined/readme.md
Outdated
Show resolved
Hide resolved
...eploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
Outdated
Show resolved
Hide resolved
...eploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
Outdated
Show resolved
Hide resolved
...eploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
Outdated
Show resolved
Hide resolved
...eploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
Outdated
Show resolved
Hide resolved
...eploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
Outdated
Show resolved
Hide resolved
@ykuijs I have modified the DRG and regenerated the code Can you check it again and let me know if I missed anything please ? Changelog: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks excellent, just a small comment on the changelog entries
@ykuijs I've just fixed the changelog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pull Request (PR) description
This Pull Request (PR) fixes the following issues
FIXES #2605
FIXES #2605