Skip to content
This repository has been archived by the owner on Jun 22, 2023. It is now read-only.

Perfect translation #16

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Perfect translation #16

wants to merge 14 commits into from

Conversation

CainLuo
Copy link

@CainLuo CainLuo commented Jan 16, 2022

Improve the untranslated, errors, etc. based on the English version of the document.

@Jinxiansen
Copy link
Member

Thank you for your translation support.
This PR has a lot of content, so it may take some time to check. Please wait patiently.

@Jinxiansen
Copy link
Member

@CainLuo
Thanks again for helping to translate and sync these documents, it will help a lot of Chinese developers.
Also I've added some comments to the PR, if you have free time please check and consider revising. 🍻

@Jinxiansen
Copy link
Member

@0xTim
Hi Tim
I found that the previously merged PR did not trigger the execution of the github action script, so the translated content was not updated to the website, such as this PR: https://github.com/vapor/docs-cn/pull/15/files.
Do you know the reason?

@0xTim
Copy link
Member

0xTim commented Jan 17, 2022

@Jinxiansen looks like the GH action was triggered - https://github.com/vapor/docs-cn/runs/4508150654?check_suite_focus=true. Does it need updating? What's missing?

@CainLuo
Copy link
Author

CainLuo commented Jan 18, 2022

@CainLuo
Thanks again for helping to translate and sync these documents, it will help a lot of Chinese developers.
Also I've added some comments to the PR, if you have free time please check and consider revising. 🍻

@Jinxiansen OK, where do I see your comment? I also found some small details on the problem, is currently being revised one by one, and will submit a PR again after all the changes, please check again then, thank you ~

@Jinxiansen
Copy link
Member

@CainLuo

Thanks again for helping to translate and sync these documents, it will help a lot of Chinese developers.

Also I've added some comments to the PR, if you have free time please check and consider revising. 🍻

@Jinxiansen OK, where do I see your comment? I also found some small details on the problem, is currently being revised one by one, and will submit a PR again after all the changes, please check again then, thank you ~

Sorry, I came back late from overtime. You can see this PR in your browser and scroll down.

@Jinxiansen
Copy link
Member

@Jinxiansen looks like the GH action was triggered - https://github.com/vapor/docs-cn/runs/4508150654?check_suite_focus=true. Does it need updating? What's missing?

Sorry, Tim, I'm not familiar with this part of the script. Tanner originally added this part of the action script task.

I am a little late after working overtime today, I will check it in my spare time.

@CainLuo
Copy link
Author

CainLuo commented Jan 19, 2022

@CainLuo

Thanks again for helping to translate and sync these documents, it will help a lot of Chinese developers.

Also I've added some comments to the PR, if you have free time please check and consider revising. 🍻

@Jinxiansen OK, where do I see your comment? I also found some small details on the problem, is currently being revised one by one, and will submit a PR again after all the changes, please check again then, thank you ~

Sorry, I came back late from overtime. You can see this PR in your browser and scroll down.

@Jinxiansen Sorry, I didn't see a reply on PR, only your conversation with 0xTim. Am I not operating correctly somewhere?

@Jinxiansen Jinxiansen self-requested a review January 19, 2022 13:02
@Jinxiansen Jinxiansen added the Update Update doc. label Jan 19, 2022
Copy link
Member

@Jinxiansen Jinxiansen left a comment

Choose a reason for hiding this comment

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

Please check these comments, thank you!


Vapor Queues ([vapor/queues](https://github.com/vapor/queues)) is a pure Swift queuing system that allows you to offload task responsibility to a side worker.
Vapor Queues[vapor/queues](https://github.com/vapor/queues))是一个纯粹的Swift队列系统,允许你将任务责任卸载给一个侧翼工作者。
Copy link
Member

Choose a reason for hiding this comment

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

感谢你的翻译支持!但我不太明白这里为何用“卸载”来表述。


- [QueuesRedisDriver](https://github.com/vapor/queues-redis-driver)

Queues also has community-based drivers:
Queues也有基于社区的驱动程序。
Copy link
Member

@Jinxiansen Jinxiansen Jan 16, 2022

Choose a reason for hiding this comment

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

感谢你的翻译支持!这里应该是冒号结尾,注意单词与汉字之间的空格。

- "控制器": "basics/controllers.md"
# TODO: 提高质量
# 多数情况下只是一个代码样本,没有什么解释。
# - "控制器": "basics/controllers.md"
- "内容": "basics/content.md"
Copy link
Member

Choose a reason for hiding this comment

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

虽然关于控制器这一页仅是提供样本示例,但让我们保持它存在可以吗?给予初学者一点或许有用的参考,除非它过时了。

# APNS

Vapor的苹果推送通知服务(APNS)API使认证和发送推送通知到苹果设备变得容易。它建立在[APNSwift](https://github.com/kylebrowning/APNSwift)的基础上。

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

Choose a reason for hiding this comment

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

可以参考术语翻译约定: https://github.com/vapor/docs-cn/wiki

```

如果您在Xcode中直接编辑清单,它将会自动接收更改并在保存文件时获取新的依赖关系。否则,从终端运行`swift package resolve`来获取新的依赖关系。

Copy link
Member

Choose a reason for hiding this comment

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

我们建议使用 “你” 来代替 “您” 即可,可以参考此术语约定: https://github.com/vapor/docs-cn/wiki

!!! tip
Workers should stay running in production. Consult your hosting provider to find out how to keep long-running processes alive. Heroku, for example, allows you to specify "worker" dynos like this in your Procfile: `worker: Run run queues --scheduled`
!!! Tip
工人应该在生产中保持运行。请咨询你的主机提供商,了解如何让长期运行的进程保持活力。例如,Heroku允许你在Procfile中像这样指定"worker"动态:`worker: Run queues --scheduled`
Copy link
Member

Choose a reason for hiding this comment

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

我认为 Workers 在这里也不必翻译,你觉得呢?

}
```

Custom servers can extend `Application.Servers.Provider` for leading-dot syntax.
自定义服务器可以扩展`Application.Servers.Provider`,以获得领先的点状语法。
Copy link
Member

Choose a reason for hiding this comment

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

以获得点语法的提示 如何?


就这样,如果你正确配置了 Nginx,你可以看到你的 Vapor 应用已经可以响应 `80` 端口上的请求了,而外部网络和你的 Vapor 应用都不会感知到 Nginx 的存在
就这样了。如果Nginx配置正确,你会看到Vapor应用程序对`80`端口的请求进行响应。Nginx代理的请求和响应是不可见的
Copy link
Member

Choose a reason for hiding this comment

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

Nginx 这一章大部分内容,我觉得原先的翻译似乎更合适一些?

@@ -14,21 +14,21 @@ app.passwords.use(...)

### Bcrypt

To use Vapor's [Bcrypt API](crypto.md#bcrypt) for password hashing, specify `.bcrypt`. This is the default.
要使用Vapor的[Bcrypt API](crypto.md#bcrypt)进行密码散列,指定`.bcrypt`。这是默认的。
Copy link
Member

Choose a reason for hiding this comment

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

"这是默认值" 如何?


### SessionID创建

除非你在[你自己的`RedisSessionsDelegate`](#RedisSessionsDelegate)中实现[`makeNewID()`](https://api.vapor.codes/redis/master/Redis/RedisSessionsDelegate/#redissessionsdelegate.makeNewID())方法,否则所有[`SessionID`](https://api.vapor.codes/vapor/master/Vapor/SessionID/)值都将通过以下方式创建:
Copy link
Member

Choose a reason for hiding this comment

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

我似乎无法打开这段文本中的2个链接,你能打开吗?也许可以尝试使用相对路径。

@Jinxiansen
Copy link
Member

@CainLuo
Please recheck these comments, you should be able to see it by now.

@CainLuo
Copy link
Author

CainLuo commented Jan 27, 2022

@CainLuo
Please recheck these comments, you should be able to see it by now.

好的,等我检查完之后再次提交合并~

@Jinxiansen
Copy link
Member

@CainLuo
Hi, CainLuo.
Do you have time to process these comments? It's not urgent.
I would love to be able to merge your PR after you finish revising it.

@0xTim
Copy link
Member

0xTim commented May 16, 2022

@Jinxiansen did this make it into the main repo or do we need to copy it over?

@Jinxiansen
Copy link
Member

@0xTim
This is not included in the main repo yet, and looks like CainLuo is a little busy and may not have time to deal with this PR.
I will try to copy the correct content in the PR and submit the PR to main Repo by the end of the week.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Update Update doc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants