Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Update defender-advanced-threat-protection.md #3255

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

Open
andrewjohnporter wants to merge 3 commits into MicrosoftDocs:main
base: main
Choose a base branch
Loading
from andrewjohnporter:patch-3

Conversation

@andrewjohnporter
Copy link
Contributor

@andrewjohnporter andrewjohnporter commented Oct 14, 2022

The security.microsoft.com Onboarding portal options seem to have changed and not match what is in this doc. 2008 is now an option on it's own

 

The security.microsoft.com Onboarding portal options seem to have changed and not match what is in this doc. 2008 is now an option on it's own
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 - @andrewjohnporter, kindly commit the suggested changes. Many thanks.

 

...protection.md
Co-authored-by: JohanFreelancer9 <48568725+JohanFreelancer9@users.noreply.github.com>
Copy link
Contributor

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

Copy link
Contributor

@BalaDelli BalaDelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#sign-off

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.

@andrewjohnporter, there is one more suggested change that needs to be committed, please. Many thanks.

Copy link
Contributor

BalaDelli commented Oct 17, 2022 via email

Hope I have committed both. Please confirm. Regards Balasubramanian Delli (Bala) Prin Product Manager #bettertogether From: JohanFreelancer9 ***@***.***> Sent: Monday, October 17, 2022 9:56 AM To: MicrosoftDocs/memdocs ***@***.***> Cc: Balasubramanian Delli ***@***.***>; Assign ***@***.***> Subject: Re: [MicrosoftDocs/memdocs] Update defender-advanced-threat-protection.md (PR #3255) @JohanFreelancer9 requested changes on this pull request. @andrewjohnporter<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandrewjohnporter&data=05%7C01%7Cbaladell%40microsoft.com%7Ce3a93f8c880b41b4b09e08daaff7b223%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C638015775668826804%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=K2hc3p4gMe64nw6pK4GuYJTJZjJ0%2B%2BFskF71zL8JQ9Q%3D&reserved=0>, there is one more suggested change that needs to be committed, please. Many thanks. - Reply to this email directly, view it on GitHub<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmemdocs%2Fpull%2F3255%23pullrequestreview-1143483961&data=05%7C01%7Cbaladell%40microsoft.com%7Ce3a93f8c880b41b4b09e08daaff7b223%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C638015775668826804%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=5AlwJ3JGkDOLRYn5REg%2FDFQ3OsYt1sSKHxgswdbpZWA%3D&reserved=0>, or unsubscribe<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATTDMJVUBGSGU2BFVYV6JW3WDTINPANCNFSM6AAAAAARFBJY54&data=05%7C01%7Cbaladell%40microsoft.com%7Ce3a93f8c880b41b4b09e08daaff7b223%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C638015775668826804%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=9UdePLtw8s2Cx7MmGF%2BxvC1ICdjGREO%2FbVO1RPbzqtU%3D&reserved=0>. You are receiving this because you were assigned.Message ID: ***@***.******@***.***>>
 

...protection.md
Co-authored-by: JohanFreelancer9 <48568725+JohanFreelancer9@users.noreply.github.com>
Copy link
Contributor

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

Copy link
Contributor

JohanFreelancer9 commented Oct 17, 2022 •
edited
Loading

Thank you, @BalaDelli.

Copy link
Contributor Author

andrewjohnporter commented Oct 17, 2022 via email

Thanks to you guys too
...
________________________________ From: JohanFreelancer9 ***@***.***> Sent: Monday, October 17, 2022 2:13:23 PM To: MicrosoftDocs/memdocs ***@***.***> Cc: Andrew John Porter ***@***.***>; Mention ***@***.***> Subject: Re: [MicrosoftDocs/memdocs] Update defender-advanced-threat-protection.md (PR #3255) Thank you, @andrewjohnporter<https://github.com/andrewjohnporter>. — Reply to this email directly, view it on GitHub<#3255 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMWWHH72WZEBNMHBXL6WPUTWDVGHHANCNFSM6AAAAAARFBJY54>. You are receiving this because you were mentioned.Message ID: ***@***.***>
JohanFreelancer9 reacted with thumbs up emoji

Copy link
Contributor

#sign-off

Copy link
Contributor

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

Copy link
Contributor

@BalaDelli BalaDelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove Windows server 2012 R2 from the list as this is not supported

Copy link
Contributor

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

Copy link
Contributor

@andrewjohnporter Hi - This PR has a merge conflict. If it's still needed, can you create a new pull request? We want to make sure you get credit.

@MandiOhlinger MandiOhlinger added the doc-wont-fix Doc issue that doesn't meet the bar to fix label Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BalaDelli BalaDelli Awaiting requested review from BalaDelli

@mestew mestew Awaiting requested review from mestew

@aczechowski aczechowski Awaiting requested review from aczechowski

1 more reviewer

@JohanFreelancer9 JohanFreelancer9 JohanFreelancer9 approved these changes

Reviewers whose approvals may not affect merge requirements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /