search

youlookwhat / CloudReader

🗡️ 云阅：一款基于网易云音乐UI，使用玩Android Api，Retrofit2 + RxJava2 + Room + MVVM-databinding架构开发的Android客户端
https://youlookwhat.github.io/CloudReader
Apache License 2.0
4.94k stars 1.33k forks source link

您好，请问什么时候能升级到RxJava2.x呢？ #99

Closed wotoutengleha closed 5 years ago

youlookwhat commented 5 years ago

马上升级 一直想着手了

youlookwhat commented 5 years ago

fix https://github.com/youlookwhat/CloudReader/issues/99 @wotoutengleha 已经升级到 RxJava2.x

youlookwhat commented 5 years ago

具体代码 9c30ac90821b2d3080d8f0877135ca32b8d36537

wotoutengleha commented 5 years ago

非常感谢 ，我获取了最新代码 viewModel 和model中的网络请求代码基本大面积都是黄色警告，我的编辑器是3.2.1 the result of subscribe is not used 出现这个警告，看着很不舒服，请问是什么原因呢

youlookwhat commented 5 years ago

能给个截图吗 我这边还没看到 没有使用subscribe 应该没有影响

wotoutengleha commented 5 years ago
wx20190225-150347 2x

1551078267461 就是这种的 ViewModel中基本都有

youlookwhat commented 5 years ago

你看一下这个类有没有类似的提示：ArticleListViewModel.java

wotoutengleha commented 5 years ago
111
wotoutengleha commented 5 years ago

这个类也有的

youlookwhat commented 5 years ago

刚升级了Android Studio，是没有处理返回值 Disposable subscribe 。添加个忽略就好，可以不用处理的。

youlookwhat commented 5 years ago

先关闭了，有什么问题再开个吧