docs(cn): translate learn/removing-effect-dependencies into Chinese#1144

creamidea · 2023-04-28T08:56:48Z

@QC-L @Yucohny @awxiaoxian2020

github-actions · 2023-04-28T08:59:28Z

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

github-actions · 2023-05-11T04:02:26Z

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Yucohny

请对照翻译规范自查一遍。

src/content/learn/removing-effect-dependencies.md

Yucohny · 2023-06-20T09:24:27Z

src/content/learn/removing-effect-dependencies.md

 ```

-[Effects "react" to reactive values.](/learn/lifecycle-of-reactive-effects#effects-react-to-reactive-values) Since `roomId` is a reactive value (it can change due to a re-render), the linter verifies that you've specified it as a dependency. If `roomId` receives a different value, React will re-synchronize your Effect. This ensures that the chat stays connected to the selected room and "reacts" to the dropdown:
+[Effects “反应”响应式值](/learn/lifecycle-of-reactive-effects#effects-react-to-react-values) 因为这里的 `roomId` 是一个响应式值（它可能随重新渲染而改变），所以 linter 会验证你是否将它指定为一个依赖项。如果 `roomId` 变成不同的值，React 将重新运行你的 Effect。这可以确保聊天界面与所选房间保持一致，并把变化“反馈”给下拉菜单：


Suggested change

[Effects “反应”响应式值](/learn/lifecycle-of-reactive-effects#effects-react-to-react-values) 因为这里的 `roomId` 是一个响应式值（它可能随重新渲染而改变），所以 linter 会验证你是否将它指定为一个依赖项。如果 `roomId` 变成不同的值，React 将重新运行你的 Effect。这可以确保聊天界面与所选房间保持一致，并把变化“反馈”给下拉菜单：

[Effect “反应”响应式值](/learn/lifecycle-of-reactive-effects#effects-react-to-react-values) 因为这里的 `roomId` 是一个响应式值（它可能随重新渲染而改变），所以 linter 会验证你是否将它指定为一个依赖项。如果 `roomId` 变成不同的值，React 将重新运行你的 Effect。这可以确保聊天界面与所选房间保持一致，并把变化“反馈”给下拉菜单：

这里的意思是，Effects => Effect ?

src/content/learn/removing-effect-dependencies.md

awxiaoxian2020 · 2023-06-21T00:37:48Z

注意到你删除了任务领取，你是否还想为这篇翻译作出贡献呢？

creamidea · 2023-06-22T02:00:52Z

@awxiaoxian2020 我没有删除任务领取，不知道是哪里出错了。我按照 @Yucohny 的要求重新检查，再提交一下。

awxiaoxian2020 · 2023-06-22T02:04:58Z

目前的问题是有重复的 PR。

creamidea · 2023-06-24T05:32:29Z

目前的问题是有重复的 PR。

了解。不过当时我是在评论区留言，且在任务列表中出现明确分配开始翻译的。

我想大概是留言的隐藏机制，导致另一位作者没有搜索到，所以以为删除了🤔

src/content/learn/removing-effect-dependencies.md

yionr · 2023-06-27T23:21:25Z

src/content/learn/removing-effect-dependencies.md

+- 当你对依赖项不满意时，你需要编辑的是代码。
+- 抑制 linter 会导致非常混乱的错误，你应该始终避免它。
+- 要移除依赖项，你需要向 linter “证明”它不是必需的。
+- 如果某些代码是响应特定交互，请将该代码移至事件处理部分。


改成 “如果某些代码是为了响应特定交互，请将该代码移至事件处理程序部分” 更通顺一些

这里我改成：如果某些代码是为了响应特定交互，请将该代码移至事件处理的地方
请帮忙评估一下。

我觉得这样也挺好的，主要拗口的是前半句

src/content/learn/removing-effect-dependencies.md

creamidea · 2023-06-28T06:30:23Z

中文翻译出现了许多“依赖列表”、“依赖关系”、“依赖项”以及“依赖”等等表述，但是有的时候对应的原文是一致的，此时建议翻译后的文本保持一致。除此之外，建议再多琢磨一下有关表达，尽量让表达更加通顺。

此提议很好，我觉得可以将其都统一为“依赖”，请评估。

creamidea · 2023-06-28T07:26:08Z

抱歉，这里使用了 force-push，之前是 merge 了 main 分支的代码吗？

Yucohny · 2023-06-28T07:47:09Z

中文翻译出现了许多“依赖列表”、“依赖关系”、“依赖项”以及“依赖”等等表述，但是有的时候对应的原文是一致的，此时建议翻译后的文本保持一致。除此之外，建议再多琢磨一下有关表达，尽量让表达更加通顺。

此提议很好，我觉得可以将其都统一为“依赖”，请评估。

可以的。

Yucohny · 2023-06-28T07:47:54Z

抱歉，这里使用了 force-push，之前是 merge 了 main 分支的代码吗？

是的，与 main 分支保持同步。

creamidea · 2023-06-28T08:13:32Z

抱歉，这里使用了 force-push，之前是 merge 了 main 分支的代码吗？

是的，与 main 分支保持同步。

了解，我现在点击了同步 main 的按钮。

creamidea marked this pull request as draft April 28, 2023 08:57

vercel bot deployed to Preview – zh-hans-react-dev April 28, 2023 08:57 View deployment

creamidea marked this pull request as ready for review April 28, 2023 09:01

creamidea force-pushed the doc/learn/removing-effect-dependencies branch from 6ccaced to 42254a3 Compare April 28, 2023 09:12

vercel bot deployed to Preview – zh-hans-react-dev April 28, 2023 09:12 View deployment

vercel bot deployed to Preview – zh-hans-reactjs-org April 28, 2023 09:12 View deployment

docs(cn): 翻译 learn/removing-effect-dependencies

55e5559

creamidea force-pushed the doc/learn/removing-effect-dependencies branch from 42254a3 to 55e5559 Compare April 28, 2023 09:16

vercel bot deployed to Preview – zh-hans-reactjs-org April 28, 2023 09:17 View deployment

vercel bot deployed to Preview – zh-hans-react-dev April 28, 2023 09:17 View deployment

Merge branch 'main' into doc/learn/removing-effect-dependencies

9c45c17

vercel bot deployed to Preview – zh-hans-reactjs-org May 11, 2023 03:59 View deployment

vercel bot deployed to Preview – zh-hans-react-dev May 11, 2023 03:59 View deployment

Yucohny mentioned this pull request Jun 19, 2023

docs(cn) translate Learn/removing effect dependencies into Chinese #1194

Closed

Yucohny reviewed Jun 20, 2023

View reviewed changes

Yucohny added the Pending Modify 已校对，待修改阶段 label Jun 20, 2023

Yucohny closed this Jun 21, 2023

awxiaoxian2020 reopened this Jun 22, 2023

Yucohny mentioned this pull request Jun 22, 2023

Translation Progress Assignment Claims #602

Open

Merge branch 'reactjs:main' into doc/learn/removing-effect-dependencies

8e37a50

vercel bot deployed to Preview – zh-hans-reactjs-org June 24, 2023 05:39 View deployment

vercel bot deployed to Preview – zh-hans-react-dev June 24, 2023 05:39 View deployment

creamidea added 2 commits June 24, 2023 06:10

doc: 按照要求处理加粗标点和属性翻译的问题

38a6dc9

doc: 处理译者注的信息

56eff16

yionr reviewed Jun 27, 2023

View reviewed changes

creamidea added 4 commits June 28, 2023 14:46

fix: 你的 Effect 和 dependency 翻译

3435078

fix: “依赖项”和“一个”翻译问题

b8c1459

fix: 链接标点符号问题

1951564

fix: internal 翻译问题

d1913b7

creamidea force-pushed the doc/learn/removing-effect-dependencies branch from 546d7aa to d1913b7 Compare June 28, 2023 07:18

vercel bot deployed to Preview – zh-hans-reactjs-org June 28, 2023 07:18 View deployment

vercel bot deployed to Preview – zh-hans-react-dev June 28, 2023 07:18 View deployment

creamidea requested review from Yucohny and yionr June 28, 2023 07:26

Merge branch 'main' into doc/learn/removing-effect-dependencies

69835c4

vercel bot deployed to Preview – zh-hans-reactjs-org June 28, 2023 08:13 View deployment

vercel bot deployed to Preview – zh-hans-react-dev June 28, 2023 08:13 View deployment

Yucohny added Pending Review 已翻译，待校对阶段 and removed Pending Modify 已校对，待修改阶段 labels Jun 28, 2023

docs(cn): review and update

3052466

vercel bot deployed to Preview – zh-hans-react-dev June 28, 2023 11:53 View deployment

vercel bot deployed to Preview – zh-hans-reactjs-org June 28, 2023 11:53 View deployment

docs(cn): review and update

ccdb432

vercel bot deployed to Preview – zh-hans-react-dev June 28, 2023 11:54 View deployment

vercel bot deployed to Preview – zh-hans-reactjs-org June 28, 2023 11:54 View deployment

Merge branch 'main' into doc/learn/removing-effect-dependencies

9b927a7

Yucohny merged commit ae69ac9 into reactjs:main Jun 28, 2023

vercel bot deployed to Preview – zh-hans-reactjs-org June 28, 2023 11:55 View deployment

vercel bot deployed to Preview – zh-hans-react-dev June 28, 2023 11:55 View deployment

Conversation

creamidea commented Apr 28, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Apr 28, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Size changes

📦 Next.js Bundle Analysis for react-dev

Uh oh!

github-actions bot commented May 11, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Size changes

📦 Next.js Bundle Analysis for react-dev

Uh oh!

Yucohny left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Yucohny Jun 20, 2023

Choose a reason for hiding this comment

Uh oh!

creamidea Jun 24, 2023

Choose a reason for hiding this comment

Uh oh!

Yucohny Jun 24, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

awxiaoxian2020 commented Jun 21, 2023

Uh oh!

creamidea commented Jun 22, 2023

Uh oh!

awxiaoxian2020 commented Jun 22, 2023

Uh oh!

creamidea commented Jun 24, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

yionr Jun 27, 2023

Choose a reason for hiding this comment

Uh oh!

creamidea Jun 28, 2023

Choose a reason for hiding this comment

Uh oh!

yionr Jun 28, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

creamidea commented Jun 28, 2023

Uh oh!

creamidea commented Jun 28, 2023

Uh oh!

Yucohny commented Jun 28, 2023

Uh oh!

Yucohny commented Jun 28, 2023

Uh oh!

creamidea commented Jun 28, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

creamidea commented Apr 28, 2023 •

edited

Loading

github-actions bot commented Apr 28, 2023 •

edited

Loading

github-actions bot commented May 11, 2023 •

edited

Loading

creamidea commented Jun 24, 2023 •

edited

Loading