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

[Beta] docs: translate README for Beta #1070

Merged
merged 18 commits into from
Mar 2, 2023
Merged

[Beta] docs: translate README for Beta #1070

merged 18 commits into from
Mar 2, 2023

Conversation

fwqaaq
Copy link
Contributor

@fwqaaq fwqaaq commented Feb 23, 2023

@awxiaoxian2020

  • 暂时只在 beta 的 README 的基础上,对 Github 的流程进行了阐述,然后并根据本地化做了一些更改,但是还有一些问题
  • react 中文文档的排版术语表,该项直接引用到中文翻译指南段落下吧。
  • 对于 LISENCE 凭证我们是否引用本仓库的凭证

beta/README.md Outdated Show resolved Hide resolved
@fwqaaq fwqaaq marked this pull request as ready for review February 23, 2023 17:17
@github-actions
Copy link

github-actions bot commented Feb 23, 2023

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

This PR introduced no changes to the javascript bundle 🙌

Copy link
Collaborator

@awxiaoxian2020 awxiaoxian2020 left a comment

Choose a reason for hiding this comment

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

请参照根目录 README,将原始文本放在最下面,Licenses 不翻译

beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
beta/README.md Outdated Show resolved Hide resolved
@awxiaoxian2020 awxiaoxian2020 linked an issue Feb 24, 2023 that may be closed by this pull request
@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

请参照原有 README,将原始文本放在最下面,Licenses 不翻译

🤔,这不妥,这会导致大量的冗余以及 markdown 语法会有大量的报错,应该移除原文档才对

@awxiaoxian2020
Copy link
Collaborator

请参照原有 README,将原始文本放在最下面,Licenses 不翻译

🤔,这不妥,这会导致大量的冗余以及 markdown 语法会有大量的报错,应该移除原文档才对

正因为本地化和原文本有一定的差异,所以还是建议保留原始文本以备查阅。Markdown 格式报错问题亦可参照仓库根目录 Readme 进行一些调整

@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

请参照原有 README,将原始文本放在最下面,Licenses 不翻译

🤔,这不妥,这会导致大量的冗余以及 markdown 语法会有大量的报错,应该移除原文档才对

正因为本地化和原文本有一定的差异,所以还是建议保留原始文本以备查阅。Markdown 格式报错问题亦可参照仓库根目录 Readme 进行一些调整

我觉得可以给一个原文地址的链接🔗,如果真将原文一坨放在页面之下,不太好

@awxiaoxian2020
Copy link
Collaborator

请参照原有 README,将原始文本放在最下面,Licenses 不翻译

🤔,这不妥,这会导致大量的冗余以及 markdown 语法会有大量的报错,应该移除原文档才对

正因为本地化和原文本有一定的差异,所以还是建议保留原始文本以备查阅。Markdown 格式报错问题亦可参照仓库根目录 Readme 进行一些调整

我觉得可以给一个原文地址的链接🔗,如果真将原文一坨放在页面之下,不太好

参考根目录的情况……就是说这个现状已经维持很久了。我们在没有合适或者说确切的理由去改变的时候,作出这样的改变可能是不恰当的。我们似乎还要考虑仓库的同步问题。

@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023 via email

@awxiaoxian2020
Copy link
Collaborator

@fwqaaq 不移除的原因可以参见这个提交的 README 同步情况
可以发现,在保留英文文本的情况下,bot 可以自动追踪变化,但是如果移除相关文本,同步机制可能会引发更多冲突。
在文档的翻译上,有关段落的更改冲突本身就是我们需要处理的,因为是一对一的翻译。但是 README 文档是不一样的。

另外 README 文档不会经常作出大的变更,所以机器人直接同步会更好一点。

还有关于这个问题的讨论,如果真的要详细讨论的话,应该另开一个 issue,并且在仓库的根目录的 README 作出变更之后再作更改 :)

@awxiaoxian2020 awxiaoxian2020 changed the title [Beta] docs: translate READEME for zh-cn.beta.react [Beta] docs: translate READEME for Beta Feb 24, 2023
@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

@fwqaaq 不移除的原因可以参见这个提交的 README 同步情况。 可以发现,在保留英文文本的情况下,bot 可以自动追踪变化,但是如果移除相关文本,同步机制可能会引发更多冲突。 在文档的翻译上,有关段落的更改冲突本身就是我们需要处理的,因为是一对一的翻译。但是 README 文档是不一样的。

另外 README 文档不会经常作出大的变更,所以机器人直接同步会更好一点。

还有关于这个问题的讨论,如果真的要详细讨论的话,应该另开一个 issue,并且在仓库的根目录的 README 作出变更之后再作更改 :)

这实在是一个两难的抉择,那是否可以使用 markdown 的注释包裹英文

fwqaaq and others added 2 commits February 24, 2023 12:06
Co-authored-by: Xavi Lee <awxiaoxian2020@163.com>
Co-authored-by: Xavi Lee <awxiaoxian2020@163.com>
@awxiaoxian2020
Copy link
Collaborator

我作了一些更改。

其实我个人更倾向于9-13行不以链接到下面的方式,而是直接放在下面。如果可以,请继续调整标题和内容结构。如果不合适,这样也是可以的。

但请重写9-13行,细化相关流程。

@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

我作了一些更改。

其实我个人更倾向于9-13行不以链接到下面的方式,而是直接放在下面。如果可以,请继续调整标题和内容结构。如果不合适,这样也是可以的。

但请重写9-13行,细化相关流程。

流程应该是很详细了,可能需要增加一些注意事项

@awxiaoxian2020
Copy link
Collaborator

我作了一些更改。
其实我个人更倾向于9-13行不以链接到下面的方式,而是直接放在下面。如果可以,请继续调整标题和内容结构。如果不合适,这样也是可以的。
但请重写9-13行,细化相关流程。

流程应该是很详细了,可能需要增加一些注意事项

我的意思是不合适,比如说“提交”一般指的是 commit

@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

行,我再想想

@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 24, 2023

不能想到更好的标题,暂时先这样吧

@awxiaoxian2020
Copy link
Collaborator

不能想到更好的标题,暂时先这样吧

可以进一步分割,我来进行更改吧。

beta/README.md Outdated Show resolved Hide resolved
@awxiaoxian2020 awxiaoxian2020 changed the title [Beta] docs: translate READEME for Beta [Beta] docs: translate README for Beta Feb 26, 2023
Co-authored-by: fw_qaq <fwqaaq@gmail.com>
@awxiaoxian2020
Copy link
Collaborator

Mark as draft for waiting for the Beta Translations Progress issue created.

@awxiaoxian2020 awxiaoxian2020 marked this pull request as draft February 26, 2023 04:58
@fwqaaq
Copy link
Contributor Author

fwqaaq commented Feb 27, 2023

Mark as draft for waiting for the Beta Translations Progress issue created.

Ok

@awxiaoxian2020 awxiaoxian2020 merged commit 38ce81c into reactjs:main Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add translation guide for the beta version docs
3 participants