加入注意事项（已增添到英文版）

[KennethKinLum]

你所看到的是一个 Pull Request 模板。（请删除此行，并请按实际情况修改下面内容，然后提交~）

目标章节：1-js/01-getting-started/1-intro

当前上游最新 commit：*

所做更改如下：

文件名	参考上游 commit	更改（理由）
article.md	a23882d	修改部分错误
limitations.png	8f13d61	新增图片

[bemself]

@KennethKinLum 感谢 PR，PR 正文的 “目标章节” 和 “所作更改” 部分的内容没有更新，请更新一下好吗？

当前最新 EN 中对应您修改的“注意事项”， 行号貌似不符，EN 276， ZH 210，所以麻烦贴一下您参考的上游文章，谢谢~

[KennethKinLum]

@KennethKinLum 感谢 PR，PR 正文的 “目标章节” 和 “所作更改” 部分的内容没有更新，请更新一下好吗？

请问 PR 是 Pull Request 是吧？
请问那儿去找： “目标章节” 和 “所作更改” ？

当前最新 EN 中对应您修改的“注意事项”， 行号貌似不符，EN 276， ZH 210，所以麻烦贴一下您参考的上游文章，谢谢~

请问另外是要修改对应行号吗？
EN 276， ZH 210 是那儿找到呢？

原文： https://javascript.info/promise-basics

Note that this is different, and more powerful than the real life “subscription list” scenario. If the singer has already released their song and then a person signs up on the subscription list, they probably won’t receive that song. Subscriptions in real life must be done prior to the event.

Promises are more flexible. We can add handlers any time: if the result is already there, our handlers get it immediately.

[bemself]

@KennethKinLum 是的， Pull Ruquest,

请问那儿去找： “目标章节” 和 “所作更改”

这里应该是您修改的文章和地址

请问另外是要修改对应行号吗？
EN 276， ZH 210 是那儿找到呢？

不好意思没说清楚，我是比对了英文最新的，发现目前的中文译版没有对应更新到最新。谢谢您添加出处，您译的这段貌似已是最新。

通常我们参考的英文是这里： en.javascript.info/article.md at master · javascript-tutorial/en.javascript.info

[bemself]

校对完成，感谢翻译~

[javascript-translate-bot]

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

[leviding]

@KennethKinLum 请尽快根据审阅者意见对 PR 的相关信息规范性，以及提交的内容进行修改。

若长时间未进行修改，则关闭处理。

[KennethKinLum]

请问已经可以了吗？我还需要处理什么吗？

[KennethKinLum]

@KennethKinLum 请尽快根据审阅者意见对 PR 的相关信息规范性，以及提交的内容进行修改。

若长时间未进行修改，则关闭处理。

请问已经可以了吗？我还需要处理什么吗？

[bemself]

请问已经可以了吗？我还需要处理什么吗？

@KennethKinLum 您这边翻译有改动吗？如有请提交一下改动；然后根据这个提示标记完成；如果对建议的翻译有不同意见，欢迎指出一起讨论，谢谢~

[KennethKinLum]

/done

[KennethKinLum]

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

/done

[bemself]

@KennethKinLum 还是没看到您新改动的代码提交，如果您本地代码改动了，得 git commit/push 到您的KennethKinLum:patch-1分支才行，麻烦您再查看一下，谢谢！

[KennethKinLum]

@KennethKinLum 还是没看到您新改动的代码提交，如果您本地代码改动了，得 git commit/push 到您的KennethKinLum:patch-1分支才行，麻烦您再查看一下，谢谢！

我不太明白。所以改动了的内容，不是在 https://github.com/javascript-tutorial/zh.javascript.info/pull/569/files 就可以完全看到，可以 merge 进了吗？

KennethKinLum wants to merge 1 commit into javascript-tutorial:master from KennethKinLum:patch-1

英文版我也是这样啊，fork 一个 repo，然后改我的 copy ，然后 Pull Request，他们就 merge 入了。中文我是完全一样的操作啊。现在到 https://github.com/KennethKinLum/zh.javascript.info/blob/patch-1/1-js/11-async/02-promise-basics/article.md 不是已经显示了 「可以注意的是：这和一个歌手的订阅列表不同」这内容吗？这代表 patch-1已经是 commit 了，已经有这个内容啊

[bemself]

@KennethKinLum

加入注意事项（已增添到英文版） by KennethKinLum · Pull Request #569 · javascript-tutorial/zh.javascript.info 这里目前看到的还是您提交的最初的原版。

想先跟您确认一下： PR 的 review comments， 您这边有采纳并在分支patch-1做了相应修改吗？

如果已采纳并修改，请提交并 push 到您仓库的这个分支；具体操作与您平时的代码提交一样。

如何验证您的更改已提交？

• 加入注意事项（已增添到英文版） by KennethKinLum · Pull Request #569 · javascript-tutorial/zh.javascript.info 这个页面应该显示您已经更改过的内容。
• Changes requested 这个状态会变为"review needed"

[KennethKinLum]

@KennethKinLum

加入注意事项（已增添到英文版） by KennethKinLum · Pull Request #569 · javascript-tutorial/zh.javascript.info 这里目前看到的还是您提交的最初的原版。

想先跟您确认一下： PR 的 review comments， 您这边有采纳并在分支patch-1做了相应修改吗？

如果已采纳并修改，请提交并 push 到您仓库的这个分支；具体操作与您平时的代码提交一样。

如何验证您的更改已提交？

• 加入注意事项（已增添到英文版） by KennethKinLum · Pull Request #569 · javascript-tutorial/zh.javascript.info 这个页面应该显示您已经更改过的内容。
• Changes requested 这个状态会变为"review needed"

可以具体一点吗? 你说 PR 的 review comments? 是指那个？ EN 276， ZH 210? a23882d ? 8f13d61 ? 采纳什么？ 做什么 "相应" 的修改？

[KennethKinLum]

你是指下面这三项吗？是否一共就这两个建议？ "Promise 则更加灵活有些" 应该是 "Promise 则更加灵活些" 吧？

1-js/11-async/02-promise-basics/article.md
1-js/11-async/02-promise-basics/article.md
1-js/11-async/02-promise-basics/article.md

可以注意的是：这和一个歌手的订阅列表不同，并且更有效能：如果歌手已经发行了一首歌，粉丝才填写订阅列表，这样粉丝不会收到通知。真实生活中，订阅必须发生在事情之前，才能收到通知或结果。
需要注意的是：这和真实生活中的”订阅列表“场景不同，并且更有效能：如果歌手已经发行了一首歌之后，人们才去注册订阅列表，这样很可能收不到那首歌。真实生活中，订阅必须发生在事件之前。

但如果是 promise，即使是事情的发生之后才加 .then/catch 处理器，仍然会马上收到通知或结果。
Promise 则更加灵活有些。我们可以在任意时间添加处理器：如果结果已经在了，我们的处理器便会立即拿到这个结果。

因为你说：

校对完成，感谢翻译~

我以为我已经不需做任何事了。原来你的意思是

校对完成，请看下面的建议修改

[KennethKinLum]

并且，我可以直接在这儿接受建议，就完成了吗？还是我要重新去 patch-1 再作修改，又重新 Pull Request?

[bemself]

@KennethKinLum 确实是

校对完成，请看下面的建议修改

谢谢指出，以后尽量少了些误解。

并且，我可以直接在这儿接受建议，就完成了吗？还是我要重新去 patch-1 再作修改，又重新 Pull Request?

不需要重新 PR，请在 patch-1分支修改并提交。
参考：
git - Preferred Github workflow for updating a pull request after code review - Stack Overflow

[KennethKinLum]

请参看下面修改： 谢谢
https://github.com/javascript-tutorial/zh.javascript.info/pull/569/files

[bemself]

@KennethKinLum 感谢~ 新年快乐！

[KennethKinLum]

谢谢你，新年快乐！

[CLAassistant]

All committers have signed the CLA.