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

feat : login activity mvvm migration #2201

Conversation

gururani-abhishek
Copy link
Contributor

@gururani-abhishek gururani-abhishek commented Jul 7, 2023

MOBILE-67

Please Add Screenshots If there are any UI changes.

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

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

-> modified Ui state class
-> added unit tests for LoginViewModel
-> updated LoginViewModel unit tests
-> added unit tests for functions related to login feature in UserAuthRepositoryTest
@gururani-abhishek gururani-abhishek marked this pull request as ready for review July 9, 2023 13:24
-> created ClientRepository, its implementation and unit tests.
-> removed Client related methods from UserAuthRepository, and its unit tests.
-> injected ClientRepositoryImp to LoginViewModel through Hilt, and added its entry to repository module of DI
-> injected preferences helper dependency through hilt
…tivity-mvvm-migration

# Conflicts:
#	app/src/main/java/org/mifos/mobile/injection/module/RepositoryModule.kt
#	app/src/main/java/org/mifos/mobile/repositories/ClientRepository.kt
#	app/src/main/java/org/mifos/mobile/repositories/ClientRepositoryImp.kt
#	app/src/main/java/org/mifos/mobile/repositories/UserAuthRepository.kt
#	app/src/main/java/org/mifos/mobile/repositories/UserAuthRepositoryImp.kt
#	app/src/test/java/org/mifos/mobile/repositories/ClientRepositoryImpTest.kt
@gururani-abhishek
Copy link
Contributor Author

Hey @jawidMuhammadi, do have a look at this whenever you find the bandwith. I've resolved the conflicts as well. 😃

…tivity-mvvm-migration

# Conflicts:
#	app/src/main/java/org/mifos/mobile/injection/module/RepositoryModule.kt
#	app/src/main/java/org/mifos/mobile/repositories/UserAuthRepository.kt
#	app/src/main/java/org/mifos/mobile/repositories/UserAuthRepositoryImp.kt
#	app/src/test/java/org/mifos/mobile/repositories/UserAuthRepositoryImpTest.kt
@jawidMuhammadi jawidMuhammadi merged commit c2e3ed4 into openMF:mvvm_architecture Jul 22, 2023
3 checks passed
@gururani-abhishek gururani-abhishek deleted the login-activity-mvvm-migration branch August 1, 2023 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants