-
Notifications
You must be signed in to change notification settings - Fork 984
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
Updating the "outdated antimalware client" alert description #4341
base: main
Are you sure you want to change the base?
Conversation
We must emphasize that this alert fires against specific Windows Defender Antimalware Platform versions. Now customers get the impression that this alert fires against Defender definition version, not platform.
@Yupiter2005 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 00fe822:
|
File | Status | Preview URL | Details |
---|---|---|---|
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md | Details |
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md
- Line 90, Column 106: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-US/microsoft-365/security/defender-endpoint/microsoft-defender-antivirus-updates?view=o365-worldwide#product-updates' contains locale code 'en-US'. For localizability, remove 'en-US' from links to most Microsoft sites.
- Line 90, Column 106: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-US/microsoft-365/security/defender-endpoint/microsoft-defender-antivirus-updates?view=o365-worldwide#product-updates' will be broken in isolated environments. Replace with a relative link.
- Line 90, Column 106: [Suggestion: preserve-view-not-set - See documentation]
You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-US/microsoft-365/security/defender-endpoint/microsoft-defender-antivirus-updates?view=o365-worldwide#product-updates
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
Few comments on the flow: Code that generates alerts of this kind: EPMGR/spGenerateAntimalwareClientVersionAlert See also: [TEST CASE 2951026] |
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md
Outdated
Show resolved
Hide resolved
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.
Copy review - @Yupiter2005, kindly commit the suggested changes. Many thanks.
Co-authored-by: JohanFreelancer9 <[email protected]>
Learn Build status updates of commit b73c43c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit cd0b835: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 869ef59: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit b1f11d5: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit b1f11d5: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit 9c1f417: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
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.
Approved
#sign-off |
@Yupiter2005 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
We must emphasize that this alert fires against specific Windows Defender Antimalware Platform versions.
Now customers get the impression that this alert fires against Defender definition version, not platform.