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.

Sign up for GitHub

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

Jump to bottom

[ar] Translate home index to arabic #45031

Merged
merged 1 commit into from
May 5, 2024
Merged

[ar] Translate home index to arabic #45031

merged 1 commit into from
May 5, 2024

Conversation

mboukhalfa
Copy link
Member

@mboukhalfa mboukhalfa commented Feb 5, 2024 •
edited

This PR adds the minimal translation related to the file /home/_index.md and splitting the old PR to smaller

Part of #44682

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Feb 5, 2024
@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Feb 5, 2024
@mboukhalfa mboukhalfa changed the base branch from main to dev-1.29-ar.1 February 5, 2024 21:46
@netlify Netlify
Copy link

netlify bot commented Feb 5, 2024

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 9225cdf
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/65c157184f9d0b000804ed56
😎 Deploy Preview https://deploy-preview-45031--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

seokho-son
seokho-son reviewed Feb 6, 2024
View reviewed changes
content/ar/docs/home/_index.md Outdated
Comment on lines 2 to 3
approvers:
- mboukhalfa
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
approvers:
- mboukhalfa
# approvers:
# - chenopis

Please do not change the names of reviewers or approvers in individual documents.
Instead, please disable the section by toggling a line comment.

@seokho-son
Copy link
Member

/language ar

@k8s-ci-robot k8s-ci-robot added the language/ar Issues or PRs related to Arabic language label Feb 6, 2024
@seokho-son
Copy link
Member

/retitle [ar] Translate home index to arabic

Please provide language code as the prefix of the PR title. (this is one of the agreed conventions from SIG Docs localization subproject meeting.)

@k8s-ci-robot k8s-ci-robot changed the title Translate home index to arabic [ar] Translate home index to arabic Feb 6, 2024
@seokho-son
Copy link
Member

For the Arabic localization (for the initial minimum-viable Arabic localization), please try to refer #44682

@seokho-son seokho-son mentioned this pull request Feb 6, 2024
Open
16 tasks
@mboukhalfa mboukhalfa force-pushed the mboukhalfa-ar-translate-home-index branch from 9225cdf to 329eb01 Compare February 6, 2024 06:58
@mboukhalfa mboukhalfa mentioned this pull request Feb 6, 2024
Open
seifrajhi
seifrajhi reviewed Apr 18, 2024
View reviewed changes
content/ar/docs/home/_index.md Outdated Show resolved Hide resolved
content/ar/docs/home/_index.md Outdated
title: "توثيق"
weight: 20
post: >
<p>تعرف على كيفية استخدام كاوبرناتيز مع الوثائق والدروس التعليمية والمراجع. يمكنك حتى أن <a href="/editdocs/" data-auto-burger-exclude data-proofer-ignore>تساعد في المساهمة في الوثائق</a>!</p>
Copy link
Member

Choose a reason for hiding this comment

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

Same for all occurrences of كاوبرناتيز to be changed to كوبيرنيتيس

content/ar/docs/home/_index.md Outdated Show resolved Hide resolved
content/ar/docs/home/_index.md
description: "تابع البرامج التعليمية لمعرفة كيفية نشر التطبيقات في كاوبرناتيز."
button: "أنظر البرامج التعليمية"
button_path: "/docs/tutorials"
- name: setup
Copy link
Member

Choose a reason for hiding this comment

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

إعداد

Copy link
Member Author

Choose a reason for hiding this comment

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

the names are kept in english : https://github.com/kubernetes/website/blob/main/content/ko/docs/home/_index.md?plain=1

content/ar/docs/home/_index.md Outdated
button: "أنظر البرامج التعليمية"
button_path: "/docs/tutorials"
- name: setup
title: "أنصب عنقود ك8ز"
Copy link
Member

Choose a reason for hiding this comment

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

إعداد عنقود ك8س

Copy link
Member Author

Choose a reason for hiding this comment

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

أفضل أنصب لاننا نستخدم أعداد مع configuration

Copy link
Member

Choose a reason for hiding this comment

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

ok makes sense, can we use ك8س ? instead of ك8ز

@mboukhalfa mboukhalfa force-pushed the mboukhalfa-ar-translate-home-index branch 5 times, most recently from 5bbb8a7 to 79e875f Compare April 20, 2024 09:00
@mboukhalfa mboukhalfa force-pushed the mboukhalfa-ar-translate-home-index branch from 79e875f to 6c25fc5 Compare May 1, 2024 01:06
@netlify Netlify
Copy link

netlify bot commented May 1, 2024 •
edited

✅ Deploy Preview for kubernetes-io-ar-staging ready!

Name Link
🔨 Latest commit d03a6fc
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-ar-staging/deploys/66374cb6e2a8e3000827e732
😎 Deploy Preview https://deploy-preview-45031--kubernetes-io-ar-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mboukhalfa mboukhalfa requested a review from seifrajhi May 3, 2024 23:45
@mboukhalfa mboukhalfa mentioned this pull request May 4, 2024
Merged
seifrajhi
seifrajhi suggested changes May 5, 2024
View reviewed changes
content/ar/docs/home/_index.md Outdated
button: "أنظر البرامج التعليمية"
button_path: "/docs/tutorials"
- name: setup
title: "أنصب عنقود ك8ز"
Copy link
Member

Choose a reason for hiding this comment

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

ok makes sense, can we use ك8س ? instead of ك8ز

@mboukhalfa
Translate home index to arabic
d03a6fc 
Signed-off-by: Mohammed Boukhalfa <mohammed.boukhalfa@est.tech>
@mboukhalfa mboukhalfa force-pushed the mboukhalfa-ar-translate-home-index branch from 6c25fc5 to d03a6fc Compare May 5, 2024 09:09
@mboukhalfa
Copy link
Member Author

@seifrajhi تم

@seifrajhi
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 5, 2024
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 3c9245c61f5dbd40b46e044b83cf47d21b56440f

@seifrajhi
Copy link
Member

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: seifrajhi

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 5, 2024
@k8s-ci-robot k8s-ci-robot merged commit 5310fc1 into kubernetes:dev-1.29-ar.1 May 5, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seifrajhi seifrajhi seifrajhi requested changes

@nate-double-u nate-double-u Awaiting requested review from nate-double-u

@windsonsea windsonsea Awaiting requested review from windsonsea

@seokho-son seokho-son Awaiting requested review from seokho-son

Assignees

@seifrajhi seifrajhi

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/ar Issues or PRs related to Arabic language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mboukhalfa @seokho-son @seifrajhi @k8s-ci-robot