-
Notifications
You must be signed in to change notification settings - Fork 531
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: update Intune assignment logic - Fixes 3892 and 3921 #3947
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.
One small comment
@William-Francillette Unit tests are failing. Could you please have a look and fix them? |
@ykuijs all sorted now |
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
Quick one, will any of the upcoming fixes help with configuration drift failing due to policy ID number in regards to IntuneDeviceConfigurationHealthMonitoringConfigurationPolicyWindows10? See below Event Viewer log:
|
@Cyanic-Cloud #3893 will fix the id issue Thanks |
Pull Request (PR) description
This Pull Request (PR) fixes the following issues
FIXES #3892
FIXES #3892
FIXES #3892
FIXES #3892
FIXES #3921
FIXES #3921