-
Notifications
You must be signed in to change notification settings - Fork 26
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
jetpack-compose-migration: ReadHistory, FeaturedReading, FeaturedDua, QuranSolution, QuranEtiquette, FeatureProphets & QuranScience #442
Conversation
Can you look at the view models here. |
Sorry, I could not get time :( I'll review and merge it soon. |
I am currently unable to work on migration due to lack of time. |
Great work! I'll take a look. |
I have fixed some UI issues.
|
No worries, I'll look into this. Ramadan Mubarak! |
Would you like to contribute to alfaazplus/SunnahApp? The app is still in development. |
Sorry, I couldn't see your edit. The app was still in development, there could be some issue loading the home screen. |
I migrated to Jetpack Compose the readHistory section of the home screen. The only problem is when deleting or adding readhistory elements, the section doesn't reflect the updated data because it needs a viewModel. I let you do that and the room database so that I can follow your approach in the future migrations that I'll do. About the color theme of the app for now I reused the already defined xml.