-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: app.controller.foo instead of 'foo' #942
Conversation
@dead-horse, thanks for your PR! By analyzing the history of the files in this pull request, we identified @leoner, @huacnlee and @tomatoo to be potential reviewers. |
和业务方聊的时候他们也不太能理解直接传一个字符串,再加上中间有一堆的 middleware 也不能穿字符串,更加容易混乱。 |
Codecov Report
@@ Coverage Diff @@
## master #942 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 28 28
Lines 669 669
=====================================
Hits 669 669 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quickstart 也改下?中英文
docs/source/zh-cn/basics/router.md
Outdated
}; | ||
|
||
// app/router/admin.js | ||
module.exports = app => { | ||
app.get('/admin/user', 'admin.user'); | ||
app.get('/admin/log', 'admin.log'); | ||
app.get('/admin/user', app.controlleradmin.user); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
少了个点
quickstart 也改了 |
+1 |
Checklist
npm test
passesAffected core subsystem(s)
Description of change
closes #920