Skip to content

Conversation

@v-maudel
Copy link
Contributor

Hello, @mairaw ,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.
"This is an instance of source issue since links for products that have migrated to markdig should be written in the form. Nonetheless, this link reported as missing in the targets does not redirect to any URL, thus it should be escaped"

Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR then share your PR number so we can confirm and close this PR.

Many thanks in advance.

Hello, @Xansky ,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.  
"This is an instance of source issue since links for products that have migrated to markdig should be written in the []() form. Nonetheless, this link reported as missing in the targets does not redirect to any URL, thus it should be escaped"

Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR  then share your PR number so we can confirm and close this PR.

Many thanks in advance.
Copy link
Contributor

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

Thanks for making this correction, @v-maudel. I'll merge your PR now.

@rpetrusha rpetrusha merged commit 778f543 into dotnet:master Jun 29, 2018
@v-maudel v-maudel deleted the patch-4 branch June 29, 2018 16:05
@v-maudel
Copy link
Contributor Author

Thanks @rpetrusha

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants