Yorkie Makoto

Results 298 comments of Yorkie Makoto

我也添加到了 major 标签,这个 PR 合并后,Release 版本需要变更为:2.0.0 > I also labeled this PR as major, after this, the next release will be 2.0.0 :)

> @yorkie 关于被微信移除掉的 registerAppWithDescription 方法,我想提出一个兼容性方案,为了不影响正在使用本库的用户,我们可以保留 registerAppWithDescription 接口,然后把它指向 SDK 的 registerApp 方法,并且给出一个 console.warn 警告用户它已经被移除掉。你觉得这样做可以吗? 我觉得没问题,不过我记得 Node.js 有专用的 deprecation 方法,不确定 ReactNative 是否也有类似方法。

Android 部分的代码有修改过,做了一些简单的抽象,iOS 的暂时没做。 On Wed, Oct 16, 2019, 9:26 AM snowfox wrote: > @yorkie 请问你这边修改 JavaScript 接口代码的时候,有修改过 > Object-c 和 Java 方面的代码吗?我似乎没看到这方面的修改记录。 > > — > You are receiving this...

> @yorkie 如果只是修改了 JavaScript 层面的代码,这个库将没法跑起来,因为核心的地方是 object-c 和 java 方面的代码, JavaScript 层的接口方法修改后需要到 IOS 和 Android 目录实现相应功能。。。 https://github.com/yorkie/react-native-wechat/commit/307ed4e9c822febe8e3e1ca7b2f2e3403a9d2193#diff-f5be6a637f5cc216645c25e9e3402450L61 可以看这里

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

嗯,可以放在根文档处 On Wed, Oct 16, 2019, 7:34 PM Kevin wrote: > *@M1seRy* commented on this pull request. > ------------------------------ > > In ios/README.txt > > : > > > @@...

@M1seRy @little-snow-fox 已经修复了上述问题,请帮忙再看一下哦。

直接显示在页面上是不是不太好,他那个应该是给应用去用的吧?

好的,我有空把 iOS 的代码也更新上去,辛苦了,用 console.warn 也可以。 On Sun, Oct 20, 2019, 10:48 AM wuxinhua wrote: > 直接显示在页面上是不是不太好,他那个应该是给应用去用的吧? > > 就是 RN 一个的 Warning 提示功能(如下图),这块我觉得可以先用 console.warn 吧。 > > 代码没有发现有其它什么问题。 > >...

@little-snow-fox @M1seRy iOS + Android + JS 部分代码都更新完毕,请再帮忙看看哈。