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

[ACA] [323630] Add path-based routing how to and conceptual. #125879

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

Conversation

v-jaswel
Copy link
Contributor

PR review team: please do not review or merge this PR until craigshoemaker signs off. Thank you!

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 22bb5de:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ⚠️Warning Details
articles/container-apps/TOC.yml ⚠️Warning Details
articles/container-apps/networking.md ✅Succeeded

articles/container-apps/how-to-use-path-based-routing.md

  • Line 56, Column 117: [Warning: file-not-found - See documentation] Invalid file link: '../articles/container-apps/whats-new.md'.
  • Line 155, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 186, 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: /cli/azure/containerapp/env/http-route-config?view=azure-cli-latest

articles/container-apps/TOC.yml

  • Line 358, Column 17: [Warning: file-not-found - See documentation] Invalid file link: 'how-to-use-path-based-rousing.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.

For any questions, please:

@v-dirichards
Copy link
Contributor

@craigshoemaker

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 PR review team label Feb 20, 2025
Copy link
Contributor

Learn Build status updates of commit 8082aac:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ⚠️Warning Details
articles/container-apps/TOC.yml ⚠️Warning Details
articles/container-apps/networking.md ✅Succeeded

articles/container-apps/how-to-use-path-based-routing.md

  • Line 56, Column 117: [Warning: file-not-found - See documentation] Invalid file link: '../articles/container-apps/whats-new.md'.
  • Line 155, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 263, 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: /cli/azure/containerapp/env/http-route-config?view=azure-cli-latest

articles/container-apps/TOC.yml

  • Line 358, Column 17: [Warning: file-not-found - See documentation] Invalid file link: 'how-to-use-path-based-rousing.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.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit d1ff6b1:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ⚠️Warning Details
articles/container-apps/TOC.yml ⚠️Warning Details
articles/container-apps/networking.md ✅Succeeded

articles/container-apps/how-to-use-path-based-routing.md

  • Line 56, Column 117: [Warning: file-not-found - See documentation] Invalid file link: '../articles/container-apps/whats-new.md'.
  • Line 155, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 272, 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: /cli/azure/containerapp/env/http-route-config?view=azure-cli-latest

articles/container-apps/TOC.yml

  • Line 358, Column 17: [Warning: file-not-found - See documentation] Invalid file link: 'how-to-use-path-based-rousing.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.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit cc1bcab:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ⚠️Warning Details
articles/container-apps/TOC.yml ⚠️Warning Details
articles/container-apps/networking.md ✅Succeeded

articles/container-apps/how-to-use-path-based-routing.md

  • Line 56, Column 117: [Warning: file-not-found - See documentation] Invalid file link: '../articles/container-apps/whats-new.md'.
  • Line 155, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).
  • Line 272, 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: /cli/azure/containerapp/env/http-route-config?view=azure-cli-latest

articles/container-apps/TOC.yml

  • Line 358, Column 17: [Warning: file-not-found - See documentation] Invalid file link: 'how-to-use-path-based-rousing.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.

For any questions, please:

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit f4dd444:

✅ Validation status: passed

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ✅Succeeded
articles/container-apps/networking.md ✅Succeeded
articles/container-apps/TOC.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

Copy link
Contributor

Learn Build status updates of commit f7a6a98:

✅ Validation status: passed

File Status Preview URL Details
articles/container-apps/how-to-use-path-based-routing.md ✅Succeeded
articles/container-apps/networking.md ✅Succeeded
articles/container-apps/TOC.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Yaml File(s) This PR includes changes to .yml file(s) owned by another author.
File Change Percent This PR contains file(s) with more than 30% file change.

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.

3 participants