-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
[hi] Add content/hi/docs/setup/learning-environment/_index.md #29416
[hi] Add content/hi/docs/setup/learning-environment/_index.md #29416
Conversation
Welcome @Darshnadas! |
✔️ Deploy Preview for kubernetes-io-main-staging ready! 🔨 Explore the source changes: af87620 🔍 Inspect the deploy log: https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/61268deb9fd8e30007fc4a96 😎 Browse the preview: https://deploy-preview-29416--kubernetes-io-main-staging.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Darshnadas I suspect that you might have misunderstood the advice to localizers in the original page.
There is a Netlify redirect from https://kubernetes.io/docs/setup/learning-environment/ to https://kubernetes.io/docs/tasks/tools/
This English page content only exists to provide a navigation stub and to protect in case that redirect is one day removed.
As you're localizing that page, you only need to copy the front matter (the first 4 lines) and add a redirect into /static/_redirects
:
- redirect
/hi/docs/setup/learning-environment/
to https://kubernetes.io/docs/tasks/tools/
[Later,] if you also localize https://kubernetes.io/docs/tasks/tools/, you can change that redirect so that it targets https://kubernetes.io/hi/docs/tasks/tools/
@sftim Ohh, okay. |
/retitle [hi] Add content/hi/docs/setup/learning-environment/_index.md |
@Darshnadas Can you implement the requested changes. We could move towards merging this PR. |
@Darshnadas Are you still working on this PR? Can you have a look at the suggestions please. |
@anubha-v-ardhan Hi, I just checked the comments and will work on the PR. |
Hi @anubha-v-ardhan I'll implement the changes and update it by tomorrow |
af87620
to
b31d559
Compare
@anubha-v-ardhan Updated the PR according to the changes, please check it. |
static/_redirects
Outdated
/pt/* /pt-br/:splat 302! | ||
|
||
/docs/tasks/tools/ /hi/docs/setup/learning-environment/ 301 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay @anubha-v-ardhan
b31d559
to
c17d383
Compare
@anubha-v-ardhan I did the required changes. |
static/_redirects
Outdated
@@ -506,6 +506,7 @@ | |||
/docs/setup/learning-environment/ /docs/tasks/tools/ 302! | |||
/id/docs/setup/learning-environment/ /id/docs/tasks/tools/ 302! | |||
/zh/docs/setup/learning-environment/ /zh/docs/tasks/tools/ 302! | |||
/hi/docs/setup/learning-environment/ /hi/docs/tasks/tools/ 302! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/hi/docs/setup/learning-environment/ /hi/docs/tasks/tools/ 302! | |
/hi/docs/setup/learning-environment/ /hi/docs/tasks/tools/ 302! |
c17d383
to
5854962
Compare
@Darshnadas Can you implement all the suggestions, Seems like only some of them were done :) |
@anubha-v-ardhan I changed the requested changes but I don't know which other areas to change or update. It would be much clearer if you point me out the places and then I would change the specific changes |
@Darshnadas The changes suggested for statics/_redirects and in #29416 (comment), #29416 (comment) and #29416 (comment). I think this happened due to making the changes manually, I would recommend to use the commit suggestion button to avoid this. It will also be easier for both you and the reviewer. Don't worry about the no. of commits :) |
/label tide/merge-method-squash If you know how to squash commits here @Darshnadas, please do. It's handy but not strictly required. |
Okay @sftim :) |
677958c
to
49a17a7
Compare
BTW, I'm still seeing multiple commits. A squash would mean there was 1 commit. |
@Darshnadas It would be nice if you can implement #29416 (comment) before squashing :) |
okay
…On Sat, Dec 18, 2021, 6:17 PM Anubhav Vardhan ***@***.***> wrote:
@Darshnadas <https://github.com/Darshnadas> It would be nice if you can
implement #29416 (comment)
<#29416 (comment)>
before squashing :)
—
Reply to this email directly, view it on GitHub
<#29416 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMOQRCROOVEDBPVR4GMLROLURR7HFANCNFSM5CGK2QNA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
okay @anubha-v-ardhan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Darshnadas You can commit the suggestion
static/_redirects
Outdated
|
||
/pt/* /pt-br/:splat 302! | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/pt/* /pt-br/:splat 302! | |
/pt/* /pt-br/:splat 302! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay
49a17a7
to
bf31565
Compare
/remove-label tide/merge-method-squash |
/assign Thank you @Darshnadas :) |
LGTM label has been added. Git tree hash: 21128c7ddb918e475e59bffa773eab72959c57a8
|
/assign @sftim |
Looks like this was squashed OK (I've been away for yule). |
lgtm from my side. @sftim If the changes look good to you, can we merge this. |
No objections from me. |
@sftim I meant can you /approve 😅 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sftim The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Thank you so much @sftim |
Localised the website/content/en/docs/setup/learning-environment/_index.md to Hindi langugage.