Skip to content

feat: settings migrate to cmp #2366

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 4 commits into from
Closed

Conversation

kapmaurya
Copy link
Contributor

@kapmaurya kapmaurya commented Apr 28, 2025

Fixes - [Jira-#Issue_Number](https://mifosforge.jira.com/browse/MIFOSAC-429)

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.

@biplab1
Copy link
Contributor

biplab1 commented Apr 28, 2025

@kapmaurya Please change the branch to kmp-impl.

@kapmaurya kapmaurya changed the base branch from development to kmp-impl April 28, 2025 15:19
Comment on lines 46 to 49
implementation(libs.androidx.ui.android)
implementation(libs.material3.android)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this block.

Comment on lines +184 to +185
uiAndroid = "1.8.0"
material3AndroidVersion = "1.3.2"
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove them as well. already present

@kapmaurya kapmaurya closed this by deleting the head repository May 15, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants