Fix mistakes in .openpublishing.redirection.json
#5451
Conversation
|
Learn Build status updates of commit 237718a: ✅ Validation status: passed
For more details, please refer to the build report. |
PRMerger Results
|
| @@ -13752,12 +13752,12 @@ | |||
| }, | |||
| { | |||
| "source_path": "docs/c-runtime-library/operator-delete-crt.md", | |||
| "redirect_url": "docs/c-runtime-library/delete-operator-crt", | |||
| "redirect_url": "/cpp/c-runtime-library/delete-operator-crt", | |||
There was a problem hiding this comment.
The old redirect_url redirects https://learn.microsoft.com/cpp/c-runtime-library/operator-delete-crt to https://learn.microsoft.com/en-us/cpp/c-runtime-library/docs/c-runtime-library/delete-operator-crt, leading to a 404.
| "redirect_document_id": false | ||
| }, | ||
| { | ||
| "source_path": "docs/c-runtime-library/operator-new-crt.md", | ||
| "redirect_url": "docs/c-runtime-library/new-operator-crt", | ||
| "redirect_url": "/cpp/c-runtime-library/new-operator-crt", |
There was a problem hiding this comment.
The old redirect_url redirects https://learn.microsoft.com/cpp/c-runtime-library/operator-new-crt to https://learn.microsoft.com/en-us/cpp/c-runtime-library/docs/c-runtime-library/new-operator-crt, leading to a 404.
| @@ -21,7 +21,7 @@ | |||
| "redirect_document_id": true | |||
| }, | |||
| { | |||
| "source_path": "docs/supported-platforms-visual-cpp.md.md", | |||
There was a problem hiding this comment.
The extra .md causes the redirection to not work (same for the other occurrence).
|
@TylerMSFT - Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
prmerger-automator
bot
added
the
aq-pr-triaged
Fix:
.mdextension insource_pathredirect_url