Integrate the new SDK to support mini program #526

little-snow-fox · 2019-10-13T16:12:22Z

使用全新的 wechat SDK，更换掉目前年份已久的SDK。

重写大部分接口，包含微信登陆、注册SDK、分享文本、分享图片、分享视频、分享网页、分享小程序、跳转到小程序等接口，并且以上接口在 IOS 和 Android 下通过简单的测试。

由于本次更新核心代码几乎重写，所以请谨慎合并，避免影响正在使用该库的用户，特别是他们的生产环境。

gdoudeng · 2019-10-14T04:39:55Z

大佬喝茶

docs/build-setup-ios.md

index.js

yorkie

@little-snow-fox 能保持与原来的接口兼容吗？这样会导致之前的应用都需要重写，多谢！

yorkie · 2019-10-14T05:02:50Z

重写大部分接口，包含微信登陆、注册SDK、分享文本、分享图片、分享视频、分享网页、分享小程序、跳转到小程序等接口，并且以上接口在 IOS 和 Android 下通过简单的测试。

能否只是在原代码基础上新增：升级 SDK、分享小程序以及跳转小程序，这样的话 PR 会更容易被合并进来。

little-snow-fox · 2019-10-14T12:28:37Z

确实，方法名称与用法的变更对已使用该库的用户影响相当大，这也是我迟迟不肯推送合并的原因。
为了与微信官方文档用法保持一致，而没沿用该库原来的方法名称和用法，确实是我考虑不周，这需要专门花时间来处理。
谢谢你抽出宝贵的时间来审查代码～

yorkie · 2019-10-14T14:09:13Z

如果你愿意，我可以抽时间在你的基础上做修改，不过需要你把我添加为你仓库的 collaborator。

…

On Mon, Oct 14, 2019, 8:28 PM snowfox ***@***.***> wrote: 确实方法名称与用法的变更对已使用该库的用户影响相当大，这也是我迟迟不肯推送合并的原因。为了与微信官方文档用法保持一致，而没沿用该库原来的方法名称和用法，确实是我考虑不周，这需要专门花时间来处理。谢谢你抽出宝贵的时间来审查代码～ — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#526?email_source=notifications&email_token=AAOYTF4YUTVSKZGAWAVBDZTQORQXNA5CNFSM4JAHSY32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBEONUA#issuecomment-541648592>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOYTF2QFI3GSOTH7JQ6TB3QORQXNANCNFSM4JAHSY3Q> .

little-snow-fox · 2019-10-14T14:23:46Z

@yorkie 太好了，很高兴你能够参与。

yorkie · 2019-10-15T08:52:10Z

@little-snow-fox @M1sery 我基于这个 PR 的基础，做了一些改动(307ed4e)主要如下：

根据现有 API 的向后兼容
重构了 Android 部分的实现，将大部分参数相关逻辑移到了 JS 层实现
基于原有基础上，添加了分享小程序与打开小程序的功能

请两位帮忙一起看看，如果没有什么问题，iOS 部分我将在这个 PR 内继续完成。

yorkie · 2019-10-15T08:54:54Z

我也添加到了 major 标签，这个 PR 合并后，Release 版本需要变更为：2.0.0

I also labeled this PR as major, after this, the next release will be 2.0.0 :)

little-snow-fox · 2019-10-15T16:17:11Z

@yorkie 关于被微信移除掉的 registerAppWithDescription 方法，我想提出一个兼容性方案，为了不影响正在使用本库的用户，我们可以保留 registerAppWithDescription 接口，然后把它指向 SDK 的 registerApp 方法，并且给出一个 console.warn 警告用户它已经被移除掉。你觉得这样做可以吗？

yorkie · 2019-10-15T18:34:33Z

@yorkie 关于被微信移除掉的 registerAppWithDescription 方法，我想提出一个兼容性方案，为了不影响正在使用本库的用户，我们可以保留 registerAppWithDescription 接口，然后把它指向 SDK 的 registerApp 方法，并且给出一个 console.warn 警告用户它已经被移除掉。你觉得这样做可以吗？

我觉得没问题，不过我记得 Node.js 有专用的 deprecation 方法，不确定 ReactNative 是否也有类似方法。

little-snow-fox · 2019-10-16T01:26:42Z

@yorkie 请问你这边修改 JavaScript 接口代码的时候，有修改过 Object-c 和 Java 方面的代码吗？我似乎没看到这方面的修改记录。

yorkie · 2019-10-16T04:24:47Z

Android 部分的代码有修改过，做了一些简单的抽象，iOS 的暂时没做。

…

On Wed, Oct 16, 2019, 9:26 AM snowfox ***@***.***> wrote: @yorkie <https://github.com/yorkie> 请问你这边修改 JavaScript 接口代码的时候，有修改过 Object-c 和 Java 方面的代码吗？我似乎没看到这方面的修改记录。 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#526?email_source=notifications&email_token=AAOYTFYRUFB4L6PHJQCSGLTQOZUVLA5CNFSM4JAHSY32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBKXB6Y#issuecomment-542470395>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOYTFYF45DQQONFDOLWPYDQOZUVLANCNFSM4JAHSY3Q> .

little-snow-fox · 2019-10-16T04:45:16Z

@yorkie 如果只是修改了 JavaScript 层面的代码，这个库将没法跑起来，因为核心的地方是 object-c 和 java 方面的代码， JavaScript 层的接口方法修改后需要到 IOS 和 Android 目录实现相应功能。。。

yorkie · 2019-10-16T05:30:15Z

@yorkie 如果只是修改了 JavaScript 层面的代码，这个库将没法跑起来，因为核心的地方是 object-c 和 java 方面的代码， JavaScript 层的接口方法修改后需要到 IOS 和 Android 目录实现相应功能。。。

307ed4e#diff-f5be6a637f5cc216645c25e9e3402450L61 可以看这里

little-snow-fox · 2019-10-16T07:49:17Z

@yorkie 但是我似乎找不到 Object-c 的改动。

yorkie · 2019-10-16T09:49:42Z

嗯，iOS 的部分我稍后会加上，我们先基于 Android 和 JS 代码的基础来讨论对外的接口，以及浇水层接口的方式是否合适 :)

yorkie · 2019-10-16T09:51:08Z

android/src/main/java/com/theweflex/react/WeChatModule.java

-            builder = builder.setResizeOptions(resizeOptions);
-        }
+        // if (resizeOptions != null) {
+        //     builder = builder.setResizeOptions(resizeOptions);


@little-snow-fox 为什么我们这里要抛弃掉原来使用的 setResizeOptions 而自己实现，我对 Android 不是很熟，所以想问一下。

因为这个方法只是对图片进行分辨率大小裁剪而已，并不是压缩大小，无法保证裁剪出来后的图片大小不超过32kb。

0xinhua · 2019-10-16T11:12:53Z

ios/README.txt

@@ -0,0 +1,114 @@
+重要!


这个 README.txt 是否可以删除，微信 SDK 的更新日志放在这感觉没有必要。

可以的，但我希望有个地方可以记录当前的 SDK 版本号。

可以的，但我希望有个地方可以记录当前的 SDK 版本号。

根目录的 README.md 里有相应的 SDK 版本的。

好的，明白。

这个问题在之后 iOS 修改后解决吧，现在大家先集中 Review 下 Android + JS 层的代码吧。

0xinhua

@little-snow-fox @yorkie 测试 Android 部分代码时发现两个问题，帮忙再 review 一下。

0xinhua · 2019-11-04T13:16:41Z

android/src/main/java/com/theweflex/react/WeChatModule.java

+        }
+
+        if (imgUrl == null) {
+            callback.invoke(null);


Suggested change

callback.invoke(null);

callback.invoke(new Object[]{});

@yorkie 这里对于非 varargs 的调用, 使用 Object[], 不然编译报错。

0xinhua · 2019-11-04T15:14:19Z

android/src/main/java/com/theweflex/react/WeChatModule.java

+            imgUrl = Uri.parse(data.getString("imageUrl"));
+            if (imgUrl.getScheme() == null) {
+                // handle static resource if no schema is provided.
+                imgUrl = getResourceDrawableURI(getReactApplicationContext(), imgUrl);


@yorkie 看一下这块代码，这里 imgUrl 是 Uri 类型，再转换为 String 类型编译会报错。

* fix: Bitmap to byte[] compile error

* fix: remove spaces and format code

0xinhua

@little-snow-fox @yorkie 这里我顺便修改了一下 bitmapTopBytes 方法。=> fix commit

android/src/main/java/com/theweflex/react/WeChatModule.java

…at into refactor-and-upgrade

yorkie · 2019-11-06T12:15:34Z

最新的提交里修改了一些 iOS 上的错误，不过还有一些库依赖问题。

* fix: format comments

0xinhua · 2019-11-08T15:55:09Z

android/src/main/java/com/theweflex/react/WeChatModule.java

+            message.messageExt = data.getString("messageExt");
+        }
+        if (thumb != null) {
+            if (thumb.length() / 1024 > THUMB_SIZE) {


这里直接使用 thumb.length() 拿不到 thumb 的大小，我明天再提个 pr 修复这个问题。

这里直接使用 thumb.length() 拿不到 thumb 的大小，我明天再提个 pr 修复这个问题。

@yorkie 已修复，在这个 PR

0xinhua

private static byte[] bitmapTopBytes(Bitmap bitmap, final boolean needRecycle) { }

这里方法名 bitmapTopBytes 拼错了，准确的命名应该是 bitmapToBytes 。

* fix: rename bitmapTopBytes => bitmapToBytesArray

snail714 · 2019-12-15T04:04:22Z

大佬们，什么时候能支持跳转小程序啊

0xinhua · 2019-12-15T05:59:51Z

@little-snow-fox @yorkie 有空找个时间一起把这个 pr 的改动再捋一遍，列个 TODO list，测试完看能不能更一版。

KingAmo · 2019-12-15T06:04:02Z

等了很久的这个PR了，大佬们辛苦了 👍

LuDongWei · 2019-12-20T02:06:19Z

大佬们辛苦了, 我们想更新这个版本，可以的话，我们可以在项目中先更新这个PR方便测试发现问题。

lp4298707 · 2019-12-26T02:16:38Z

时刻关注这个PR,感谢大佬们的贡献

wong2 · 2020-02-13T06:34:00Z

还在进行吗？

gdoudeng · 2020-02-14T04:11:14Z

还在进行吗？

我项目都写完几个了，这个pr居然还没合并

wong2 · 2020-02-14T06:54:55Z

@gdoudeng 感觉有点难了，我直接用 little-snow-fox/react-native-wechat-lib 了

wong2 · 2020-02-14T07:01:21Z

我觉得既然微信SDK都大版本更新了，这边也没必要一定要保持和之前接口的兼容，愿意更新到大版本的需要接受breaking change

gdoudeng · 2020-02-19T08:32:02Z

@gdoudeng 感觉有点难了，我直接用 little-snow-fox/react-native-wechat-lib 了

合理，新项目用它完全可以

ccksfh · 2020-03-19T10:03:44Z

这个pr流产了吗？伤心...

vance-liu · 2020-03-29T11:04:49Z

我打算搞下来自己维护了。。。伤心。。。

hengkx · 2020-05-22T03:20:32Z

还处理吗？

youngjuning · 2021-07-22T06:35:36Z

还在维护吗？

little-snow-fox mentioned this pull request Oct 13, 2019

提交苹果审核的时候出现警告，ios13开始使用 webview 的应用不再允许提交 #521

Open

yorkie reviewed Oct 14, 2019

View reviewed changes

docs/build-setup-ios.md Outdated Show resolved Hide resolved

yorkie reviewed Oct 14, 2019

View reviewed changes

docs/build-setup-ios.md Outdated Show resolved Hide resolved

yorkie reviewed Oct 14, 2019

View reviewed changes

index.js Outdated Show resolved Hide resolved

yorkie reviewed Oct 14, 2019

View reviewed changes

index.js Outdated Show resolved Hide resolved

yorkie suggested changes Oct 14, 2019

View reviewed changes

yorkie mentioned this pull request Oct 14, 2019

fix interface ShareMetadata #527

Merged

little-snow-fox and others added 2 commits October 15, 2019 12:51

🔨 Use the new wechat SDK, rewrite most interface methods.

b06293a

refactor the android part and backward to the current api

307ed4e

yorkie force-pushed the master branch from e222668 to 307ed4e Compare October 15, 2019 08:47

yorkie added the major label Oct 15, 2019

This was referenced Oct 15, 2019

feature: 并入github.com/puti94添加的小程序功能 #504

Closed

修复0.60以上ios打包问题 #497

Closed

yorkie reviewed Oct 16, 2019

View reviewed changes

0xinhua reviewed Oct 16, 2019

View reviewed changes

0xinhua reviewed Nov 4, 2019

View reviewed changes

0xinhua added 2 commits November 5, 2019 21:03

fix: Bitmap to byte[] compile error (#2)

aab1b63

* fix: Bitmap to byte[] compile error

fix: remove spaces and format code (#3)

a03c52f

* fix: remove spaces and format code

0xinhua reviewed Nov 5, 2019

View reviewed changes

android/src/main/java/com/theweflex/react/WeChatModule.java Outdated Show resolved Hide resolved

yorkie reviewed Nov 6, 2019

View reviewed changes

android/src/main/java/com/theweflex/react/WeChatModule.java Outdated Show resolved Hide resolved

yorkie reviewed Nov 6, 2019

View reviewed changes

android/src/main/java/com/theweflex/react/WeChatModule.java Show resolved Hide resolved

yorkie added 2 commits November 6, 2019 20:10

fixes ios errors

cba9ca2

Merge branch 'master' of github.com:little-snow-fox/react-native-wech…

77a8465

…at into refactor-and-upgrade

yorkie force-pushed the master branch from a03c52f to 307ed4e Compare November 6, 2019 12:13

chore: formatting code comments

dff761a

* fix: format comments

0xinhua reviewed Nov 8, 2019

View reviewed changes

0xinhua reviewed Nov 17, 2019

View reviewed changes

fix: bitmapTopBytes => bitmapToBytesArray (#5)

3ba9a48

* fix: rename bitmapTopBytes => bitmapToBytesArray

0xinhua mentioned this pull request Dec 13, 2019

微信sdk更新,平台预计从2020年3月起，逐步回收旧版本SDK的能力. #538

Open

0xinhua mentioned this pull request Dec 15, 2019

请问下，能支持 RN 拉起微信小程序功能吗？ #539

Open

Integrate the new SDK to support mini program #526

Are you sure you want to change the base?

Integrate the new SDK to support mini program #526

Conversation

little-snow-fox commented Oct 13, 2019

gdoudeng commented Oct 14, 2019

yorkie left a comment

Choose a reason for hiding this comment

yorkie commented Oct 14, 2019

little-snow-fox commented Oct 14, 2019 • edited Loading

yorkie commented Oct 14, 2019 via email

little-snow-fox commented Oct 14, 2019

yorkie commented Oct 15, 2019 • edited Loading

yorkie commented Oct 15, 2019

little-snow-fox commented Oct 15, 2019 • edited Loading

yorkie commented Oct 15, 2019 • edited by 0xinhua Loading

little-snow-fox commented Oct 16, 2019

yorkie commented Oct 16, 2019 via email

little-snow-fox commented Oct 16, 2019

yorkie commented Oct 16, 2019

little-snow-fox commented Oct 16, 2019

yorkie commented Oct 16, 2019

Choose a reason for hiding this comment

little-snow-fox Oct 16, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0xinhua left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0xinhua left a comment

Choose a reason for hiding this comment

yorkie commented Nov 6, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0xinhua left a comment • edited Loading

Choose a reason for hiding this comment

snail714 commented Dec 15, 2019

0xinhua commented Dec 15, 2019

KingAmo commented Dec 15, 2019

LuDongWei commented Dec 20, 2019

lp4298707 commented Dec 26, 2019

wong2 commented Feb 13, 2020

gdoudeng commented Feb 14, 2020

wong2 commented Feb 14, 2020

wong2 commented Feb 14, 2020

gdoudeng commented Feb 19, 2020

ccksfh commented Mar 19, 2020

vance-liu commented Mar 29, 2020

hengkx commented May 22, 2020

youngjuning commented Jul 22, 2021

little-snow-fox commented Oct 14, 2019 •

edited

Loading

yorkie commented Oct 15, 2019 •

edited

Loading

little-snow-fox commented Oct 15, 2019 •

edited

Loading

yorkie commented Oct 15, 2019 •

edited by 0xinhua

Loading

little-snow-fox Oct 16, 2019 •

edited

Loading

0xinhua left a comment •

edited

Loading