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

docs(core/cookie-and-session): translate section Cookie #1562

Merged
merged 4 commits into from
Nov 14, 2017

Conversation

zhang-z
Copy link
Contributor

@zhang-z zhang-z commented Oct 25, 2017

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Description of change

Work In Progress: Translate core/cookie-and-session

@zhang-z zhang-z mentioned this pull request Oct 25, 2017
48 tasks
@codecov-io
Copy link

codecov-io commented Oct 25, 2017

Codecov Report

Merging #1562 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1562   +/-   ##
=======================================
  Coverage   99.57%   99.57%           
=======================================
  Files          29       29           
  Lines         712      712           
=======================================
  Hits          709      709           
  Misses          3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ab0b5f1...e7da951. Read the comment docs.

@popomore popomore added the WIP label Oct 25, 2017
@popomore
Copy link
Member

Thx for the translation. If you complete the translation, you can ping me to remove the label.

@zhang-z
Copy link
Contributor Author

zhang-z commented Oct 28, 2017

@popomore , the translation of cookie-and-session is done.


```js
ctx.cookies.set(key, value, {
httpOnly: true, // 默认就是 true
Copy link
Member

Choose a reason for hiding this comment

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

translate the comments to English too


Session is built on top of Cookie.
By default, the content of Session is stored in a Cookie field as encrypted string.
Every time client sends requests to server, is cookie is attached.
Copy link
Member

Choose a reason for hiding this comment

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

is cookie is attached 不太通顺?

@zhang-z
Copy link
Contributor Author

zhang-z commented Nov 13, 2017

@dead-horse , updated according to your comment.

@dead-horse dead-horse merged commit 49b0071 into eggjs:master Nov 14, 2017
@dead-horse
Copy link
Member

LGTM

@zhang-z zhang-z deleted the feature/docs-core-cookie-and-session branch November 14, 2017 02:31
popomore pushed a commit that referenced this pull request Nov 14, 2017
docs(core/cookie-and-session): translate section Cookie (#1562)
@atian25 atian25 removed the WIP label Nov 14, 2017
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.

5 participants