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

Upgrade to AndroidX #31

Merged
merged 5 commits into from
Jan 19, 2023
Merged

Upgrade to AndroidX #31

merged 5 commits into from
Jan 19, 2023

Conversation

deadman96385
Copy link
Contributor

@deadman96385 deadman96385 commented Jan 4, 2023

Fixes #30

I used the automated converter, which worked overall other then I had to also get an updated copy of Expandable Recycler. This was needed because AndroidX changes the Recyclerview some and so the support library you use needed updates. I've tested this against both the your sample app as well as the expandable recyclerview sample app. I haven't noticed any breakages.

https://github.com/deadman96385/expandable-recycler-view/commits/master

Copy link
Owner

@AliAsadi AliAsadi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thank you for making those changes. 🙌🏼

Left some minor questions, let me know what do you think.

build.gradle Outdated Show resolved Hide resolved
power-preference/build.gradle Show resolved Hide resolved
Used the automated conversion in Android Studio. We need to move to a newer version of Expandable Recyclerview which will happen in the next commit.
The original library is abandoned, someone else already updated it but added their own stuff. I've only kept the AndroidX upgrade releated changes from them. I then updated Gradle to the latest in the version order that Android Studio updated in.

New Upstream Fork:
https://github.com/AliAsadi/expandable-recycler-view

Original Fork:
https://github.com/deadman96385/expandable-recycler-view
Removed the applicationid from the manifest and converted the library and sample app to use Namespace as needed by newer gradle. Application id is recommended to match namespace. Which you can see below:

https://developer.android.com/studio/build/configure-app-module#set-namespace:~:text=application%20ID.-,Change%20the%20namespace,-In%20most%20cases

Update gradle-wrapper.properties
Copy link
Owner

@AliAsadi AliAsadi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@AliAsadi AliAsadi merged commit 0f935ce into AliAsadi:master Jan 19, 2023
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.

AndroidX Conversion
2 participants