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: fix reference framework path #634

Merged
merged 7 commits into from
Mar 24, 2017
Merged

docs: fix reference framework path #634

merged 7 commits into from
Mar 24, 2017

Conversation

kylezhang
Copy link
Contributor

  • The framework.md docs path changed, this reference docs didn't change。
Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

kylezhang and others added 5 commits March 17, 2017 11:33
`docs/source/zh-cn/core/httpclient.md`:HttpClient 默认全局配置,笔误修改多余的`POST => PUT`
`docs/source/zh-cn/core/unittest.md`:漏了 `过`
-  The framework.md docs path changed, this reference docs didn't change。
@mention-bot
Copy link

@kylezhang, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dead-horse, @huacnlee and @simman to be potential reviewers.

@@ -187,7 +187,7 @@ exports.login = function* (ctx) {

// 设置 Session
this.session.user = user;
// 如果用户勾选了 `记住我`,设置一个月的过期时间
// 如果用户勾选了 `记住我`,设置一分钟的过期时间
Copy link
Member

Choose a reason for hiding this comment

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

这个地方之前已经改过了, 是改下面的.

你的代码基线太旧了, rebase 下最新的 master

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

-  The framework.md docs path changed, this reference docs didn't change。
@atian25 atian25 merged commit 7d830b7 into eggjs:master Mar 24, 2017
@codecov
Copy link

codecov bot commented Mar 24, 2017

Codecov Report

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

@@          Coverage Diff          @@
##           master   #634   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          26     26           
  Lines         633    633           
=====================================
  Hits          633    633

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 a471e93...961572b. Read the comment docs.

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.

3 participants