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

refactor: Migrate LoanAccountSummary fragment to compose #2124

Merged

Conversation

itsPronay
Copy link
Contributor

@itsPronay itsPronay commented Jul 2, 2024

Fixes JIRA: https://mifosforge.jira.com/browse/MIFOSAC-184

XML (before) Compose (now)
image image
Screencast_07_03_2024_03.20.36_AM.webm

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

  • Apply the MifosStyle.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.

itsPronay and others added 8 commits July 3, 2024 04:17
* refactor: refactor About activity to compose

* fix : added text style
* refactor: refactor Create new center fragment to compose

* fix: fix build fail
migrate dataTable to compose

topbar fix

removed extra file

fix topbar

made required changes
…into loan_account_summary_migration

# Conflicts:
#	core/data/src/main/java/com/mifos/core/data/di/DataModule.kt
#	core/designsystem/src/main/java/com/mifos/core/designsystem/theme/Color.kt
#	core/designsystem/src/main/java/com/mifos/core/designsystem/theme/MifosTextStyle.kt
#	feature/client/src/main/res/values/strings.xml
#	mifosng-android/src/main/java/com/mifos/mifosxdroid/injection/module/RepositoryModule.kt
@itsPronay itsPronay changed the title refactor: Migrate Loan account summary to compose refactor: Migrate LoanAccountSummary fragment to compose Jul 2, 2024
@itsPronay itsPronay requested a review from Aditya-gupta99 July 3, 2024 13:12
@therajanmaurya
Copy link
Member

Give padding 24dp and 24dp from left and right.

@itsPronay itsPronay force-pushed the loan_account_summary_migration branch from 68da110 to 6943837 Compare July 4, 2024 03:05
remove extra icon

added horizontal padding 24.dp

fix scale

Fix scale
@itsPronay itsPronay force-pushed the loan_account_summary_migration branch from e589a8b to 95a8b85 Compare July 4, 2024 03:34
@itsPronay
Copy link
Contributor Author

Give padding 24dp and 24dp from left and right.

@therajanmaurya done
image

@therajanmaurya therajanmaurya merged commit 49bd77e into openMF:master Jul 7, 2024
5 checks passed
@itsPronay itsPronay deleted the loan_account_summary_migration branch July 10, 2024 04:06
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.

3 participants