Skip to content

Conversation

@P-ppc
Copy link
Contributor

@P-ppc P-ppc commented Mar 23, 2017

qq20170323-0 2x 在api中添加is_uped字段

qq20170323-1 2x 修改api文档,添加对应描述

@mention-bot
Copy link

@P-ppc, thanks for your PR! By analyzing the history of the files in this pull request, we identified @TakWolf, @alsotang and @lanceli to be potential reviewers.

@codecov-io
Copy link

codecov-io commented Mar 23, 2017

Codecov Report

Merging #934 into master will increase coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #934      +/-   ##
==========================================
+ Coverage   87.47%   87.74%   +0.27%     
==========================================
  Files          53       53              
  Lines        2331     2334       +3     
  Branches      341      342       +1     
==========================================
+ Hits         2039     2048       +9     
+ Misses        292      286       -6
Impacted Files Coverage Δ
api/v1/topic.js 89.92% <100%> (+4.63%) ⬆️

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 9c0b3fd...12002e8. Read the comment docs.

@alsotang
Copy link
Member

客户端不是本来就有用户的 id 值吗?这个本地做做判断就好了

@alsotang alsotang merged commit cc252fb into cnodejs:master Mar 23, 2017
@P-ppc P-ppc deleted the add-is_uped branch March 23, 2017 07:07
@alsotang
Copy link
Member

已发布

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