-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
update kotlin to 1.8.22, replace deprecated kotlin synthetics with jetpack bindings #902
base: master
Are you sure you want to change the base?
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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)
app/src/main/java/de/grobox/transportr/trips/search/DirectionsActivity.kt
Outdated
Show resolved
Hide resolved
4ba2cf6
to
b93ce78
Compare
app/src/main/java/de/grobox/transportr/trips/search/ProductDialogFragment.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.22
and migrates the deprecated kotlin synthetics by jetpack bindings as described hereDone
./update-dependency-pinning.sh
connectedDebugAndroidTest
🟥 fails? (is this know, i cannot comparemaster
as 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.