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.

Sign up for GitHub

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

Jump to bottom

新增会员卡相关接口 #280

Merged
merged 2 commits into from
Jul 15, 2017
Merged

新增会员卡相关接口 #280

merged 2 commits into from
Jul 15, 2017

Conversation

mgcnrx11
Copy link

  1. 激活会员卡接口
  2. 拉取会员信息接口

@mgcnrx11
新增会员卡相关接口
c56bd76 
1. 激活会员卡接口
2. 拉取会员信息接口
@mgcnrx11
Copy link
Author

mgcnrx11 commented Jul 13, 2017 •
edited
Loading

文档地址：https://mp.weixin.qq.com/wiki?t=resource/res_main&id=mp1451025283

6.1 接口激活
6.2.1 普通一键激活中的步骤5

@mgcnrx11
Copy link
Author

fail的原因貌似是sonarQube连不上服务器，这Travis CI不应去执行Sonar吧？

@binarywang
Copy link
Owner

感谢再次提供代码，CI问题可以先忽略。
说下我发现的两个问题吧，

  1. 首先新加的接口代码好像没有提供单元测试代码，能否后期补一下？
  2. 另外有个包名好像拼错了。
    weixin-java-mp/src/main/java/me/chanjar/weixin/mp/bean/mambercard/MemberCardUserInfo.java
    bean后面的mambercard应该为membercard吧

@mgcnrx11
Copy link
Author

OK👌，我再补

@binarywang binarywang merged commit 3a2efdd into binarywang:develop Jul 15, 2017
comeonc added a commit to comeonc/weixin-java-tools that referenced this pull request Jul 19, 2017
@comeonc
Merge branch 'develop' of github.com:Wechat-Group/weixin-java-tools i…
fd508b2 
…nto develop

* 'develop' of github.com:Wechat-Group/weixin-java-tools:
  发布临时测试版本2.7.6.BETA
  binarywang#178 实现查询代金券批次和信息的接口
  增加会员卡管理服务的`更新会员信息`接口的实现 (binarywang#283)
  binarywang#281 消息路由器增加对EventKey正则表达式匹配的支持
  binarywang#178 实现发送代金券接口
  新增会员卡相关接口 (binarywang#280)
  binarywang#279 统一下单接口参数对象WxPayUnifiedOrderRequest增加fingerprint属性
  update travis settings
binarywang pushed a commit that referenced this pull request Sep 2, 2017
@mgcnrx11 @binarywang
新增会员卡相关接口 (#280)
2726325 
* 新增会员卡相关接口
1. 激活会员卡接口
2. 拉取会员信息接口
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mgcnrx11 @binarywang