Skip to content

Update connectivity URLs with Microsoft 365 Unified Domains note - #585

Open
mts-2021 wants to merge 2 commits into
MicrosoftDocs:publicfrom
mts-2021:patch-1
Open

Update connectivity URLs with Microsoft 365 Unified Domains note#585
mts-2021 wants to merge 2 commits into
MicrosoftDocs:publicfrom
mts-2021:patch-1

Conversation

@mts-2021

Copy link
Copy Markdown

Since the URLs listed under Microsoft 365 Unified Domains are also required, a note regarding Microsoft 365 Unified Domains has been added, along with reference links for convenience.

Since the URLs listed under Microsoft 365 Unified Domains are also required, a note regarding Microsoft 365 Unified Domains has been added, along with reference links for convenience.
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit e1a3f7f:

⚠️ Validation status: warnings

File Status Preview URL Details
defender-endpoint/standard-device-connectivity-urls-commercial.md ⚠️Warning Details

defender-endpoint/standard-device-connectivity-urls-commercial.md

  • Line 145, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 145, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' will be broken in isolated environments. Replace with a relative link.
  • Line 145, Column 3: [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/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains

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.

@v-dirichards

Copy link
Copy Markdown
Contributor

Limor Wainstein (@limwainstein)

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged Tracking label for the vendor PR Review team label Aug 26, 2026
Comment thread defender-endpoint/standard-device-connectivity-urls-commercial.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a documentation note to the Defender portal connectivity URLs page to highlight that Microsoft 365 unified domains might also need to be allowed, and includes a reference for readers.

Changes:

  • Added a new NOTE callout under “Defender portal URLs” about Microsoft 365 unified domains.
  • Included a reference link to the Microsoft 365 unified domains section for convenience.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +142 to +145
> In addition to the URLs listed in this table, access may also need to be allowed for the URLs listed under Microsoft 365 Unified Domains.
>
> Microsoft 365 Unified Domains
> https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains
Co-authored-by: Diana Richards <103777760+v-dirichards@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit a4ea5d2:

💡 Validation status: suggestions

File Status Preview URL Details
defender-endpoint/standard-device-connectivity-urls-commercial.md 💡Suggestion Details

defender-endpoint/standard-device-connectivity-urls-commercial.md

  • Line 145, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' will be broken in isolated environments. Replace with a relative link.
  • Line 145, Column 3: [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/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains

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.

@mts-2021 mts-2021 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thank you for your review and the updates.
Could you please review it again at your convenience?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants