Update connectivity URLs with Microsoft 365 Unified Domains note - #585
Conversation
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.
|
mts-2021 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
mts-2021 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit e1a3f7f:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| defender-endpoint/standard-device-connectivity-urls-commercial.md | 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.
|
Limor Wainstein (@limwainstein) Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
There was a problem hiding this comment.
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.
| > 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 status updates of commit a4ea5d2: 💡 Validation status: suggestions
defender-endpoint/standard-device-connectivity-urls-commercial.md
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
left a comment
There was a problem hiding this comment.
Thank you for your review and the updates.
Could you please review it again at your convenience?
|
Limor Wainstein (@limwainstein) This PR looks ready to go - unless you want to touch up the formatting. |
Updated note regarding Microsoft 365 Unified Domains for clarity.
|
Learn Build status updates of commit d8d4acb: 💡 Validation status: suggestions
defender-endpoint/standard-device-connectivity-urls-commercial.md
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. |
|
Learn Build status updates of commit 3aaebf1: 💡 Validation status: suggestions
defender-endpoint/standard-device-connectivity-urls-commercial.md
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. |
|
Learn Build status updates of commit 4d09197: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Limor Wainstein (@limwainstein) I edited the note per the copilot suggestion. If this is OK, please sign off. This was one of two similar PRs. |
|
Learn Build status updates of commit 9510ed4: ✅ Validation status: passed
For more details, please refer to the build report. |
c67b323
into
MicrosoftDocs:public
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.