Skip to content

Conversation

@itsPronay
Copy link
Collaborator

Fixes - https://mifosforge.jira.com/browse/MIFOSAC-341

This pull request presents an improved version of #2300 by @biplab1 , addressing several issues identified in the original submission

biplab1 and others added 30 commits February 6, 2025 18:58
…nt-dbflow-to-room

# Conflicts:
#	core/data/src/main/java/com/mifos/core/data/repository/ClientIdentifiersRepository.kt
#	core/data/src/main/java/com/mifos/core/data/repository/CreateNewClientRepository.kt
#	core/data/src/main/java/com/mifos/core/data/repositoryImp/ClientIdentifiersRepositoryImp.kt
#	core/data/src/main/java/com/mifos/core/data/repositoryImp/CreateNewClientRepositoryImp.kt
#	core/database/src/main/java/com/mifos/room/dao/ClientDao.kt
#	core/database/src/main/java/com/mifos/room/db/MifosDatabase.kt
#	core/database/src/main/java/com/mifos/room/di/DaoModule.kt
#	core/database/src/main/java/com/mifos/room/entities/client/Savings.kt
* [database.group] dbflow to room

* fix checks
* MIFOSAC-342 [database.center] dbFlow to room migration
* feat(office): migrated Dbflow to Room

* fix: data type and removed Gson

* fix: model usage in useCase

* refactor: replaced entity instead of model

* fix: spotless error

* fix checks failure

---------

Co-authored-by: Nagarjuna <nagarjunabanda4@gmail.com>
itsPronay and others added 9 commits February 20, 2025 23:07
* MIFOSAC-392 [database/charge] dbflow to room

* backup

* Fix exception

* added singleton
# Conflicts:
#	core/data/src/main/java/com/mifos/core/data/pagingSource/ClientChargesPagingSource.kt
#	core/data/src/main/java/com/mifos/core/data/repository/CreateNewClientRepository.kt
#	core/data/src/main/java/com/mifos/core/data/repositoryImp/CreateNewClientRepositoryImp.kt
#	core/database/src/main/java/com/mifos/room/db/MifosDatabase.kt
#	core/database/src/main/java/com/mifos/room/entities/accounts/savings/SavingsAccount.kt
@itsPronay itsPronay marked this pull request as draft February 28, 2025 15:45
@itsPronay itsPronay marked this pull request as ready for review February 28, 2025 15:50
Copy link
Collaborator

@niyajali niyajali left a comment

Choose a reason for hiding this comment

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

Do this changes or if you planning to fix in other PR, let us know we will proceed with Merge.

@biplab1
Copy link
Contributor

biplab1 commented Mar 3, 2025

@itsPronay Thanks for improving on my initial work. I'm still learning Android and Kotlin, so this is helpful for me to see.

@niyajali
Copy link
Collaborator

niyajali commented Mar 6, 2025

@itsPronay Should we proceed with merge

@therajanmaurya therajanmaurya merged commit 885e878 into openMF:kmp-impl Mar 6, 2025
5 checks passed
itsPronay added a commit to itsPronay/android-client that referenced this pull request Aug 5, 2025
* refactor(core): migrate from dbflow to room in core:data and core:database

* refactor(core): replace var with val in core:database

* refactor(core): replace var with val in core:database and delete some files

* refactor(core): update files after ci-prepush

* refactor(core:database): remove columninfo annotations from room entities

* chore: apply ci-prepush changes

* refactor: correct import statements for client room entities

* chore: update ci-prepush changes

* feat(core:database): Update ClientDao

* refactor(core): migrate from dbflow to room in core:data and core:database

* refactor(core): replace var with val in core:database

* refactor(core): replace var with val in core:database and delete some files

* refactor(core): update files after ci-prepush

* refactor(core:database): remove columninfo annotations from room entities

* chore: apply ci-prepush changes

* refactor: correct import statements for client room entities

* chore: update ci-prepush changes

* feat(core:database): Update ClientDao

* refactor(core:database): remove columninfo annotaions from room:entities:templates

* refactor(core:database): replace HashMap with emptyMap

* refactor(core): restore original files DataManagerClient and ActivatePayload

* refactor(core:data): migrate dbflow to room

* refactor(core:database): update ClientDao

* refactor(core): update ClientDao and other files in :core:data and :core:database

* refactor: [database.group] dbflow to room (openMF#2313)

* [database.group] dbflow to room

* fix checks

* refactor: [database.center] dbFlow to room migration (openMF#2307)

* MIFOSAC-342 [database.center] dbFlow to room migration

* refactor: [database.office] dbflow to room (openMF#2314)

* feat(office): migrated Dbflow to Room

* fix: data type and removed Gson

* fix: model usage in useCase

* refactor: replaced entity instead of model

* fix: spotless error

* fix checks failure

---------

Co-authored-by: Nagarjuna <nagarjunabanda4@gmail.com>

* refactor: [database/charge] dbflow to room (openMF#2312)

* MIFOSAC-392 [database/charge] dbflow to room

* backup

* Fix exception

* added singleton

* refactor(core:database): update ClientDaoHelper and ClientAccounts

* refactor(core:database): update ClientDaoHelper and ClientTypeConverters

* refactor(core:database): update DAO classes and clean up comments

* [database/client] dbflow to room

* fix

* made changes

---------

Co-authored-by: Biplab Dutta <biplabdutta27@gmail.com>
Co-authored-by: Nagarjuna <nagarjunabanda4@gmail.com>
itsPronay added a commit to itsPronay/android-client that referenced this pull request Aug 5, 2025
* refactor(core): migrate from dbflow to room in core:data and core:database

* refactor(core): replace var with val in core:database

* refactor(core): replace var with val in core:database and delete some files

* refactor(core): update files after ci-prepush

* refactor(core:database): remove columninfo annotations from room entities

* chore: apply ci-prepush changes

* refactor: correct import statements for client room entities

* chore: update ci-prepush changes

* feat(core:database): Update ClientDao

* refactor(core): migrate from dbflow to room in core:data and core:database

* refactor(core): replace var with val in core:database

* refactor(core): replace var with val in core:database and delete some files

* refactor(core): update files after ci-prepush

* refactor(core:database): remove columninfo annotations from room entities

* chore: apply ci-prepush changes

* refactor: correct import statements for client room entities

* chore: update ci-prepush changes

* feat(core:database): Update ClientDao

* refactor(core:database): remove columninfo annotaions from room:entities:templates

* refactor(core:database): replace HashMap with emptyMap

* refactor(core): restore original files DataManagerClient and ActivatePayload

* refactor(core:data): migrate dbflow to room

* refactor(core:database): update ClientDao

* refactor(core): update ClientDao and other files in :core:data and :core:database

* refactor: [database.group] dbflow to room (openMF#2313)

* [database.group] dbflow to room

* fix checks

* refactor: [database.center] dbFlow to room migration (openMF#2307)

* MIFOSAC-342 [database.center] dbFlow to room migration

* refactor: [database.office] dbflow to room (openMF#2314)

* feat(office): migrated Dbflow to Room

* fix: data type and removed Gson

* fix: model usage in useCase

* refactor: replaced entity instead of model

* fix: spotless error

* fix checks failure

---------

Co-authored-by: Nagarjuna <nagarjunabanda4@gmail.com>

* refactor: [database/charge] dbflow to room (openMF#2312)

* MIFOSAC-392 [database/charge] dbflow to room

* backup

* Fix exception

* added singleton

* refactor(core:database): update ClientDaoHelper and ClientAccounts

* refactor(core:database): update ClientDaoHelper and ClientTypeConverters

* refactor(core:database): update DAO classes and clean up comments

* [database/client] dbflow to room

* fix

* made changes

---------

Co-authored-by: Biplab Dutta <biplabdutta27@gmail.com>
Co-authored-by: Nagarjuna <nagarjunabanda4@gmail.com>
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.

4 participants