-
Notifications
You must be signed in to change notification settings - Fork 664
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 #2142: migrated guarantor related fragments to viewbinding #2159
fix #2142: migrated guarantor related fragments to viewbinding #2159
Conversation
do review this @gururani-abhishek !! |
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 solid!! good job.
binding.tilFirstName.editText?.setText(payload?.firstname) | ||
binding.tilLastName.editText?.setText(payload?.lastname) | ||
binding.tilOfficeName.editText?.setText(payload?.officeName) | ||
(binding.guarantorTypeField.editText as? MaterialAutoCompleteTextView)?.setText(payload?.guarantorType?.value!!) | ||
|
||
} | ||
|
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.
It seems findPreviouslySelectedGurantorType() is not being used anywhere, can be removed after a thorough usage check.
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.
done @gururani-abhishek , I have also used with(binding)
.setMessage(getString(R.string.dialog_are_you_sure_that_you_want_to_string, | ||
getString(R.string.delete_guarantor))) | ||
.setPositiveButton(getString(R.string.yes), | ||
DialogInterface.OnClickListener { _, _ -> |
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.
This redundant SAM constructor can be reformat. Please have a look.
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.
done
Hey @jawidMuhammadi kindly review this as well |
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!
7c30f64
into
openMF:migrate_to_view_binding
Fixes #2142
vidma_recorder_04062023_001804.mp4
Please Add Screenshots If there are any UI changes.
Please make sure these boxes are checked before submitting your pull request - thanks!
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.