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: Transfer process migrated to mvvm & implemented coroutines #2248

Merged

Conversation

PratyushSingh07
Copy link
Collaborator

@PratyushSingh07 PratyushSingh07 commented Aug 1, 2023

Jira Ticket : https://mifosforge.jira.com/browse/MOBILE-74
Coroutine: https://mifosforge.jira.com/browse/MOBILE-122

  • 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.

@PratyushSingh07 PratyushSingh07 changed the title Transfer process mvvm feat: Transfer process migrated to mvvm Aug 1, 2023
@PratyushSingh07 PratyushSingh07 changed the title feat: Transfer process migrated to mvvm feat: Transfer process migrated to mvvm & implemented coroutines Aug 8, 2023
@PratyushSingh07
Copy link
Collaborator Author

This PR has a lot of commits and thus I wont be implementing the coroutine test rules here. I will make a separate PR for this after it gets merged

@jawidMuhammadi jawidMuhammadi merged commit f592f93 into openMF:development Aug 17, 2023
3 checks passed
@PratyushSingh07 PratyushSingh07 deleted the transferProcess-mvvm branch October 5, 2023 21:02
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