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

chore: Remove locales from docs.microsoft.com urls #1011

Merged
merged 2 commits into from
Nov 4, 2020

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Nov 3, 2020

Why:

Microsoft links automatically direct to the users locale when it is removed from the URLs. This is also done for the docs.microsoft.com content in order to improve localization of content. There were changes to the translated content in the cleanup, but it wasn't committed here, so it would flow through the regular translation flow.

What's being changed:

Ran the cleanup from https://github.com/microsoft/vscode-docs-authoring/ to strip locales from MS urls

Check off the following:

@nschonni nschonni requested a review from a team as a code owner November 3, 2020 04:58
Copy link
Contributor

@SMAKSS SMAKSS left a comment

Choose a reason for hiding this comment

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

Well, I tried this with some different languages by changing the browser language and it is working as expected.

Copy link
Contributor

@heiskr heiskr left a comment

Choose a reason for hiding this comment

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

Awesome, nice catch

@janiceilene
Copy link
Contributor

Thanks @nschonni! I'll triage this for a content review, as well.

@janiceilene janiceilene added the content This issue or pull request belongs to the Docs Content team label Nov 4, 2020
@nschonni nschonni force-pushed the remove-microsoft-locales branch 2 times, most recently from 5377aeb to 758429a Compare November 4, 2020 21:22
Copy link
Contributor

@rachmari rachmari left a comment

Choose a reason for hiding this comment

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

@nschonni thank you! This looks great! I'll get this merged right away. 🚀

@rachmari rachmari merged commit 87e8d1b into github:main Nov 4, 2020
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

@nschonni nschonni deleted the remove-microsoft-locales branch November 4, 2020 22:44
jnidzwetzki pushed a commit to jnidzwetzki/docs that referenced this pull request Oct 6, 2022
* Update Prometheus config

* Update promscale/send-data/prometheus.md

Co-authored-by: Lana Brindley <github@lanabrindley.com>

* Update promscale/send-data/prometheus.md

Co-authored-by: Lana Brindley <github@lanabrindley.com>

Co-authored-by: Lana Brindley <github@lanabrindley.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants