增加了用户对其它用户的屏蔽功能 #373
Closed
增加了用户对其它用户的屏蔽功能 #373
Conversation
This file contains hidden or 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
controllers/sign.js
Outdated
Member
There was a problem hiding this comment.
我正在看这个 pr，这个地方其实应该写 user.toObject({virtual: true}) 的，因为这样一来可以把 avatar_url 也提取出来。
Member
There was a problem hiding this comment.
不过这部分的逻辑我已经改过了，所以 .toObject 就用不到了。
Member
There was a problem hiding this comment.
用户之间的关注功能取消了对吧？我也觉得那东西可以取消了。
card.html 那里还有些东西可以删，也是关于关注和被关注的。
我得好好看看这个 pr，影响比较多。
Contributor
Author
There was a problem hiding this comment.
没取消，侧栏里的card已经有关注和取消按钮了，这里再有就重复了
Contributor
Author
There was a problem hiding this comment.
我觉得关注可以保留着，以后基于这个可以做点功能
Member
|
屏蔽用户的功能。。这个。。我觉得吧，加个禁言功能就好了，把 kingapple 禁言就好了。 |
Contributor
Author
|
同意，加这项功能纯粹就为了屏蔽这货 |
Member
|
谢谢哈。真的很不好意思让你白写了那么多代码，老实说你的代码质量不错，测试也有。：） |
Contributor
Author
|
目的已经达到就行咯：） |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
RT