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

Migrate from RxJava2 to RxJava3 for the whole project and migrate to AndroidX for sample app #152

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mmBs
Copy link

@mmBs mmBs commented Dec 27, 2020

Hi,

I migrated the lib to RxJava3 as RxJava2 will be deprecated soon (end of support is the last day of Dec 2020).
I tested the lib on the sample app which I also adapted to the newest dependencies (AndroidX etc.)
I haven't rewritten it to Kotlin but I will consider it if I have some time ;)

I hope you will find this PR helpful.

Cheers,
Michal

With bump of compileSdk, build tools, targetSdk and version of the code.
Also, fix the findConventions taks and replace << with doLast.
When the library was updated to RxJava3, it was required to refresh
the sample as well. The refactor consists of:

- bumping comple, target sdk and build tools
- adding sourceCompatibility and targetCompatibility
- refreshinng dependecies for RxJava (2 --> 3)
- refreshing dependencies for PermissionsDispatcher and RxPermissions
- adding dependencies for AndroidX (annotation and app compat)
- removing some deprecated methods (like getFragmentManager())
- adapting the code to new APIs from RxPermissions
and PermissionsDispatcher
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

Successfully merging this pull request may close these issues.

1 participant