docs(controller): import base controller directly #1771
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dead-horse
force-pushed
the
docs-controller-customize
branch
from
2a9d870
to
6c0444f
Compare
|
这样啊，如果很多级的 Controller，就要写很长的相对路径，感觉挺不 egg-style 的 |
|
@atian25 虽然填写路径麻烦些，但是这样更灵活，不同的controller可以选择继承不同的基类，声明controller时，也更加清晰到底继承了哪些类。 |
Codecov Report
@@ Coverage Diff @@
## master #1771 +/- ##
=======================================
Coverage 99.58% 99.58%
=======================================
Files 29 29
Lines 729 729
=======================================
Hits 726 726
Misses 3 3
Continue to review full report at Codecov.
|
atian25
reviewed
Dec 1, 2017
| ``` | ||
|
|
||
| Now we can use base class' methods when defining Controllers for the application: | ||
| Now we can use base class' methods by extends from `BaseController`: |
There was a problem hiding this comment.
这个就是这样的吧。其他地方语法错误更多，留待有缘人来改善吧。英语太差。
atian25
approved these changes
Dec 1, 2017
popomore
pushed a commit
that referenced
this pull request
Dec 1, 2017
docs(controller): import base controller directly (#1771)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
npm testpassesAffected core subsystem(s)
Description of change