-
Notifications
You must be signed in to change notification settings - Fork 539
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
Update network-topology-and-connectivity guidance with Hybrid connect… #1324
base: main
Are you sure you want to change the base?
Conversation
…ivity details using Express Routes
@aarthiem : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit e80fc24:
|
File | Status | Preview URL | Details |
---|---|---|---|
docs/scenarios/app-platform/api-management/network-topology-and-connectivity.md | Details |
docs/scenarios/app-platform/api-management/network-topology-and-connectivity.md
- Line 28, Column 157: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/azure/expressroute/designing-for-high-availability-with-expressroute' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
- Line 28, Column 157: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/azure/expressroute/designing-for-high-availability-with-expressroute' will be broken in isolated environments. Replace with a relative link.
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
@MicrosoftDocs/patterns-and-practices-team-pr-reviewers
NOTE: The validation issues around the links will need to be addressed before we can merge. (hint: remove #label:"aq-pr-triaged" |
#label:"pending-content-team/business-approval" |
This pull request has been inactive for 14 days. If you are finished with your changes, don't forget to sign off. For instructions, see the contributor guide. |
…ivity details using Express Routes
Before selecting the "Create pull request" button:
Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.
Describe the summary, scope, and intent of this PR:
As part of the overall resiliency program, we are adding details about resiliency best practices on Express Routes for hybrid connectivity.
Insert links(s) to any related work item(s) or supporting detail: https://dev.azure.com/CSUSolEng/Azure%20Infra%20Networking/_workitems/edit/32583
AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors
Additional guidance for private repos and internal contributors
@MicrosoftDocs/cloud-architecture-content-team-pr-reviewers
in your PR and someone will get back to you. After CACT review is complete, aCACT #sign-off
will be added.@MicrosoftDocs/cloud-architecture-content-team-pr-reviewers