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

#1645 fixed the error in the EditPasswordScreen.kt regarding the Mif… #1647

Merged
merged 1 commit into from
Jun 5, 2024
Merged

#1645 fixed the error in the EditPasswordScreen.kt regarding the Mif… #1647

merged 1 commit into from
Jun 5, 2024

Conversation

Sarthak9504
Copy link
Contributor

Issue Fix

Fixes #1646

Description

The EditPasswordScreen.kt had some errors related to the composable MifosScaffold.k where the scaffold content was not
being passed as one of the arguments required by the Composable. The error was fixed along with teh help of
@PratyushSingh07.

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

…osScaffold.kt Composable

 Issues Fix
 Fixes #1646

 Description
 The EditPasswordScreen.kt had some errors related to the composable MifosScaffold.k where the scaffold content was not being passed as one of the arguments required by the Composable. The error was fixed along with teh help of @PratyushSingh07.
@therajanmaurya
Copy link
Member

Add video of before and after

@Sarthak9504
Copy link
Contributor Author

Add video of before and after

Actually It was not an UI issue there was an error in that file the project was not building due to it. One of the parameters of the function used was missing which was causing the error.

@therajanmaurya
Copy link
Member

@PratyushSingh07 review

@PratyushSingh07
Copy link
Collaborator

Already reviewed @therajanmaurya . I had connected with him over a call and then he had made these changes in my presence

@therajanmaurya therajanmaurya merged commit 16c3ee8 into openMF:dev Jun 5, 2024
2 of 4 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.

Error while building the application
3 participants