Yorkie Makoto

Results 298 comments of Yorkie Makoto

@lolBig may I ask you to use `Buffer` instead of `String`, the former is in usual in Node.js ecosystem :)

@lolBig The CI in node stable seems failed.

Ok, #71 fixes the problem, so I will merge this after it gets done :)

@mwpnl @Southern after taking a look at gdb results, I think the potential missing line is [here](https://github.com/Southern/node-x509/blob/master/src/x509.cc#L281): ``` cc char *data = (char*) malloc(bptr->length + 1); BUF_strlcpy(data, bptr->data, bptr->length +...

Okay @mwpnl will comment when I get home later, thanks :)

By the way, please follow the coding style at first.

PR is always welcome, @lambrojos

> 重写大部分接口,包含微信登陆、注册SDK、分享文本、分享图片、分享视频、分享网页、分享小程序、跳转到小程序等接口,并且以上接口在 IOS 和 Android 下通过简单的测试。 能否只是在原代码基础上新增:升级 SDK、分享小程序以及跳转小程序,这样的话 PR 会更容易被合并进来。

如果你愿意,我可以抽时间在你的基础上做修改,不过需要你把我添加为你仓库的 collaborator。 On Mon, Oct 14, 2019, 8:28 PM snowfox wrote: > 确实方法名称与用法的变更对已使用该库的用户影响相当大,这也是我迟迟不肯推送合并的原因。 > 为了与微信官方文档用法保持一致,而没沿用该库原来的方法名称和用法,确实是我考虑不周,这需要专门花时间来处理。 > 谢谢你抽出宝贵的时间来审查代码~ > > — > You are receiving this because you commented. > Reply...

@little-snow-fox @M1seRy 我基于这个 PR 的基础,做了一些改动(307ed4e9c822febe8e3e1ca7b2f2e3403a9d2193)主要如下: - 根据现有 API 的向后兼容 - 重构了 Android 部分的实现,将大部分参数相关逻辑移到了 JS 层实现 - 基于原有基础上,添加了分享小程序与打开小程序的功能 请两位帮忙一起看看,如果没有什么问题,iOS 部分我将在这个 PR 内继续完成。