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

Updating the "outdated antimalware client" alert description #4341

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Yupiter2005
Copy link
Member

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.

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.
Copy link
Contributor

@Yupiter2005 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 00fe822:

⚠️ Validation status: warnings

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ⚠️Warning 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:

@Yupiter2005
Copy link
Member Author

Few comments on the flow:

Code that generates alerts of this kind: EPMGR/spGenerateAntimalwareClientVersionAlert
The list of "expired" versions is provided via AdminUIContent file: CAMPServicingStates.xml

See also: [TEST CASE 2951026]

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a 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 <48568725+JohanFreelancer9@users.noreply.github.com>
Copy link
Contributor

Learn Build status updates of commit b73c43c:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit cd0b835:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 869ef59:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit b1f11d5:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit b1f11d5:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 9c1f417:

✅ Validation status: passed

File Status Preview URL Details
memdocs/configmgr/protect/deploy-use/endpoint-configure-alerts.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants