update kotlin to 1.8.22, replace deprecated kotlin synthetics with jetpack bindings#902
update kotlin to 1.8.22, replace deprecated kotlin synthetics with jetpack bindings#902mesinger wants to merge 3 commits into
Conversation
|
Thank you for your pull request and welcome to our community! We require contributors to sign our Contributor License Agreement, and we don't seem to have the user @mesinger on file. In order for your code to get reviewed and merged, please explicitly state that you accept the agreement. Alternatively, you can add a commit that adds yourself to https://github.com/grote/Transportr/blob/master/.clabot |
|
Hey quick side context why i did this PR, I really like Transportr and there is a specific feature I'd love it to have, so i checked the repo/issues and jadajadajada... So today I tried to setup my local dev environment and this PR was the result of it. ... Furthermore I was thinking of the possibility of contributing to this app, despite the fact of knowing the infrequent development the last couple of years and investing time in the onboarding of a dev to a project that lacks invested time anyway is 💀. As @Altonss and @ialokim you seem to be the most recent contributors, see any sense in further cotributing to Transportr? My Background: Greetings EDIT: The feature I'd like to have in Transportr mentioned above, i did a poc here. Feel free to give your feedback to that also. |
Hello @mesinger,
Personnaly I find Transportr really useful and would be happy to see this software maintained in the future :) Some things just require basic maintenance, while some new features would require major developments. For now I focus on maintenance, but I'm open for more bigger changes :) |
|
Happy to hear that. Do you have any way to communicate besides github e.g. Discord? |
39e1176 to
353f759
Compare
353f759 to
4ba2cf6
Compare
|
ping: please review this pr |
Altonss
left a comment
There was a problem hiding this comment.
Thanks a lot for this PR, sorry for the late review.
Transportr is crashing with this error:
java.lang.NullPointerException: Parameter specified as non-null is null: method androidx.lifecycle.ViewModelProvider.<init>, parameter factory
at androidx.lifecycle.ViewModelProvider.<init>(Unknown Source:7)
at de.grobox.transportr.trips.search.ProductDialogFragment.onCreateView(ProductDialogFragment.java:80)
at androidx.fragment.app.Fragment.performCreateView(Fragment.java:2600)
at androidx.fragment.app.FragmentManagerImpl.moveToState(FragmentManagerImpl.java:881)
at androidx.fragment.app.FragmentManagerImpl.moveFragmentToExpectedState(FragmentManagerImpl.java:1238)
at androidx.fragment.app.FragmentManagerImpl.moveToState(FragmentManagerImpl.java:1303)
at androidx.fragment.app.BackStackRecord.executeOps(BackStackRecord.java:439)
at androidx.fragment.app.FragmentManagerImpl.executeOps(FragmentManagerImpl.java:2079)
at androidx.fragment.app.FragmentManagerImpl.executeOpsTogether(FragmentManagerImpl.java:1869)
at androidx.fragment.app.FragmentManagerImpl.removeRedundantOperationsAndExecute(FragmentManagerImpl.java:1824)
at androidx.fragment.app.FragmentManagerImpl.execPendingActions(FragmentManagerImpl.java:1727)
at androidx.fragment.app.FragmentManagerImpl$2.run(FragmentManagerImpl.java:150)
at android.os.Handler.handleCallback(Handler.java:942)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7940)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:952)
4ba2cf6 to
b93ce78
Compare
Altonss
left a comment
There was a problem hiding this comment.
Looks good to me :)
Tested on Android 13, and everything worked fine!
Maybe just to be sure, one could also test if it still works on older android versions (like android 5)
|
I'll try to finally test this PR on an older device before merging it :) |
Updates Kotlin to
1.8.22and migrates the deprecated kotlin synthetics by jetpack bindings as described hereDone
./update-dependency-pinning.shconnectedDebugAndroidTest🟥 fails? (is this know, i cannot comparemasteras that one isn't building on my machine)ℹ️ I did this because I tried to get the project running on my machine and some solution was bumping kotlin.