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

I18n sync learn #41061

Merged
merged 1 commit into from
Feb 16, 2021
Merged

I18n sync learn #41061

merged 1 commit into from
Feb 16, 2021

Conversation

naomi-lgbt
Copy link
Member

Checklist:

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the main branch of freeCodeCamp.
  • All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

@gitpod-io
Copy link

gitpod-io bot commented Feb 11, 2021

@RandellDawson RandellDawson changed the base branch from i18n-sync-learn-old to main February 11, 2021 21:35
Comment on lines +13 to +14
- 克隆 [this GitHub repo](https://github.com/freeCodeCamp/boilerplate-project-exercisetracker/) 并在本地完成项目。
- 使用 [repl.it 初始化项目](https://repl.it/github/freeCodeCamp/boilerplate-project-exercisetracker) 来完成项目。
Copy link
Member Author

Choose a reason for hiding this comment

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

Why is one link text translated but the other isn't?

Copy link
Member

Choose a reason for hiding this comment

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

I tagged the string on Crowdin.

challengeType: 2
forumTopicId: 301517
dashedName: serve-json-on-a-specific-route
---

# --description--

While an HTML server serves HTML, an API serves data. A <dfn>REST</dfn> (REpresentational State Transfer) API allows data exchange in a simple way, without the need for clients to know any detail about the server. The client only needs to know where the resource is (the URL), and the action it wants to perform on it (the verb). The GET verb is used when you are fetching some information, without modifying anything. These days, the preferred data format for moving information around the web is JSON. Simply put, JSON is a convenient way to represent a JavaScript object as a string, so it can be easily transmitted.
HTML 服务器提供 HTML 服务,而 API 提供数据服务。 <dfn>具象状态传输</dfn>(REST,REpresentational State TransferAPI 允许以简单的方式进行数据交换,对于客户端不必要知道服务器的细节。 客户只需要知道资源在哪里(URL),以及想执行的动作(动词)。 GET 动词常被用来获取无需修改的信息。 如今,网络上的移动数据首选格式是 JSON, 简而言之,JSON 是一种可以方便地用字符串表示 JavaScript 对象的方式,因此它很容易传输。
Copy link
Member Author

Choose a reason for hiding this comment

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

Should REST (in the definition tags) be translated?

Copy link
Member

Choose a reason for hiding this comment

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

@nhcarrigan It should not. It has been fixed on Crowdin. In an effort to get this PR merged, since I have reviewed everything else on it and think it is fine to merge, I will create an additional commit to switch it to back to REST.

Copy link
Member

Choose a reason for hiding this comment

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

I am holding off on the extra commit as it will fix itself on the next download.

@naomi-lgbt naomi-lgbt marked this pull request as ready for review February 16, 2021 20:39
@naomi-lgbt naomi-lgbt closed this Feb 16, 2021
@naomi-lgbt naomi-lgbt reopened this Feb 16, 2021
@gitpod-io
Copy link

gitpod-io bot commented Feb 16, 2021

@naomi-lgbt
Copy link
Member Author

Not sure where the CI went

Copy link
Member

@RandellDawson RandellDawson left a comment

Choose a reason for hiding this comment

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

LGTM

@RandellDawson RandellDawson merged commit 785405f into main Feb 16, 2021
@RandellDawson RandellDawson deleted the i18n-sync-learn branch February 16, 2021 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants