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: Migrating database module from dbflow to room db #2286

Merged
merged 5 commits into from
Jan 17, 2025

Conversation

Darkeye14
Copy link
Contributor

adding entities for the migration. here we have added entities for accounts package

Fixes - Jira-#278

Didn't create a Jira ticket, click here to create new.

Please Add Screenshots If there are any UI changes.

Before After

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

  • Run the static analysis check ./gradlew check or ci-prepush.sh to make sure you didn't break anything

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

adding entities for the migration. here we have added entities for accounts package
@Darkeye14
Copy link
Contributor Author

Darkeye14 commented Jan 15, 2025

@itsPronay, just a heads up, when you push commits apply spotless and detekt everytime. Also, See if any app crashes are induced !!
Do update your local branch regularly!

Lets connect tonight whenever you are free

@Darkeye14 Darkeye14 marked this pull request as draft January 15, 2025 10:30
adding entities for the migration. here we have added entities for center, client, collection sheet,group, navigation package
due date type converter and database module cache issue solved
added entities fir the rest of the modules
migrated model package to the room related package
@Darkeye14 Darkeye14 marked this pull request as ready for review January 17, 2025 10:41
@Darkeye14
Copy link
Contributor Author

WhatsApp.Video.2025-01-17.at.16.03.09_bb099a69.mp4

@niyajali niyajali merged commit 5a80565 into openMF:kmp-impl Jan 17, 2025
5 checks passed
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