-
Notifications
You must be signed in to change notification settings - Fork 674
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
fix : migrated activities from butterknife to viewbinding #2144
fix : migrated activities from butterknife to viewbinding #2144
Conversation
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.
Please have an overall look at all the activities and if you find instances of Kotlin Synthetics then replace them with ViewBinding.
And do the same with unused imports, remove them please.
app/src/main/java/org/mifos/mobile/ui/activities/HomeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/HomeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/EditUserDetailActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/HomeActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/LoginActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/LoginActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/SavingsAccountApplicationActivity.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/mifos/mobile/ui/activities/HomeActivity.kt
Outdated
Show resolved
Hide resolved
daecc08
to
d19dea6
Compare
app/src/main/java/org/mifos/mobile/ui/activities/EditUserDetailActivity.kt
Outdated
Show resolved
Hide resolved
1df33c8
to
737ae06
Compare
Hey @jawidMuhammadi , do have a look at this PR , I have linked the issue to this PR and also changed the PR description |
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.
LGTM!
fixes #2142
Migrated Activities from butterknife to viewbinding as part of GSoC 23.
build.gradle file will be redundant if PR #2143 gets merged and in that case , I will remove it from this PR
Apply the
AndroidStyle.xml
style template to your code in Android Studio.Run the unit tests with
./gradlew check
to make sure you didn't break anythingIf you have multiple commits please combine them into one commit by squashing them.