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: 修正Controller文档中关于jsonp配置的错误 #372

Merged
merged 1 commit into from
Feb 13, 2017

Conversation

moldauder
Copy link
Contributor

原文档中 https://eggjs.org/zh-cn/basics/controller.html#jsonp-配置 的配置说明是

module.exports = {
  callback: 'callback', // 识别 query 中的 `callback` 参数
  limit: 100, // 函数名最长为 100 个字符
}

实际上应为

exports.jsonp = {...}

@mention-bot
Copy link

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

@popomore popomore changed the title 修正Controller文档中关于jsonp配置的错误 docs: 修正Controller文档中关于jsonp配置的错误 Feb 13, 2017
@dead-horse
Copy link
Member

感谢反馈,下次提交的 commit message 可以按照规范来提,里面不要出现中文。

@dead-horse dead-horse merged commit 194a47c into eggjs:master Feb 13, 2017
@atian25
Copy link
Member

atian25 commented Feb 13, 2017

@dead-horse 应该修改 PR title 后再 merge,我们现在都是 square 的

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