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

menuCreat()可能存在的问题 #80

Closed
laohezi opened this issue Nov 18, 2016 · 2 comments
Closed

menuCreat()可能存在的问题 #80

laohezi opened this issue Nov 18, 2016 · 2 comments

Comments

@laohezi
Copy link

laohezi commented Nov 18, 2016 •
edited

在实现类中的menuCreate(String agentId, WxMenu menu)方法中agentid 应该是传进来的agentId,而不是

  • this.configStorage.getAgentId();吧

@OverRide
public void menuCreate(WxMenu menu) throws WxErrorException {
menuCreate(this.configStorage.getAgentId(), menu);
}

@OverRide
public void menuCreate(String agentId, WxMenu menu) throws WxErrorException {
String url = "https://qyapi.weixin.qq.com/cgi-bin/menu/create?agentid="

+ this.configStorage.getAgentId();

post(url, menu.toJson());

}
@binarywang
Copy link
Member

看着像是有问题的，如果有时间你可以帮忙提交个PR上来。

binarywang added a commit that referenced this issue Nov 21, 2016
@binarywang
修复 #80 menuCreate()方法代码的问题，and reformat code
72d6aad
@binarywang
Copy link
Member

已修复

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@binarywang @laohezi