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.

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

updateMatieral有错 #55

Open
vectorZou opened this issue Aug 17, 2015 · 4 comments
Open

updateMatieral有错 #55

vectorZou opened this issue Aug 17, 2015 · 4 comments

Comments

@vectorZou
Copy link

exports.updateNewsMaterial = function (news, callback) {
this.preRequest(this._uploadNewsMaterial, arguments);
};

/*!

  • 新增永久图文素材的未封装版本
    */
    // 这里的地址有错/add_news?access_token应该是update_news吧
    exports._updateNewsMaterial = function (news, callback) {
    var url = this.prefix + 'material/add_news?access_token=' + this.token.accessToken;
    this.request(url, postJSON(news), wrapper(callback));
    };
@vectorZou
Copy link
Author

now i pretty sure this is a bug for updateMaterial function. is it you want to do this?

@JacksonTian
Copy link
Member

please report the api doc address. thank you.

@vectorZou
Copy link
Author

/lib/api_material.js line at 212 and 219

JacksonTian added a commit that referenced this issue Sep 29, 2015
fix更新永久图文素材 #55
@loulin
Copy link

loulin commented Sep 29, 2015

@vectorZou 可以自己提一个pr学习一下,还有英文要加强哟
有空可以领个任务,把所有的API用以下形式重新封装一下:

make(exports, 'updateNewsMaterial', function (news, callback) {
  var url = this.prefix + 'material/update_news?access_token=' + this.token.accessToken;
  this.request(url, postJSON(news), wrapper(callback));
});

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

No branches or pull requests

3 participants