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

fix : migrated activities from butterknife to viewbinding #2144

Conversation

PratyushSingh07
Copy link
Collaborator

@PratyushSingh07 PratyushSingh07 commented May 18, 2023

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 anything

  • If you have multiple commits please combine them into one commit by squashing them.

Copy link
Contributor

@gururani-abhishek gururani-abhishek left a 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.

@PratyushSingh07 PratyushSingh07 changed the title migrated activities from butterknife to viewbinding fix : migrated activities from butterknife to viewbinding May 21, 2023
@PratyushSingh07
Copy link
Collaborator Author

Hey @jawidMuhammadi , do have a look at this PR , I have linked the issue to this PR and also changed the PR description
Do let me know if there are other changes that needs to be done

Copy link
Contributor

@jawidMuhammadi jawidMuhammadi left a comment

Choose a reason for hiding this comment

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

LGTM!

@jawidMuhammadi jawidMuhammadi merged commit f9e4122 into openMF:migrate_to_view_binding May 22, 2023
@PratyushSingh07 PratyushSingh07 deleted the butterknife-viewBinding branch June 7, 2023 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants