Skip to content

Feature/mifosac 579 client fields#2517

Closed
SnehaghoshBarsha444 wants to merge 4 commits intoopenMF:developmentfrom
SnehaghoshBarsha444:feature/MIFOSAC-579-client-fields
Closed

Feature/mifosac 579 client fields#2517
SnehaghoshBarsha444 wants to merge 4 commits intoopenMF:developmentfrom
SnehaghoshBarsha444:feature/MIFOSAC-579-client-fields

Conversation

@SnehaghoshBarsha444
Copy link

Fixes - Jira-#Issue_Number->MIFOSAC-579

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.

Copilot AI and others added 3 commits October 4, 2025 07:42
Copy link
Contributor

@revanthkumarJ revanthkumarJ left a comment

Choose a reason for hiding this comment

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

and also upload a video showing what you changed and how it reflects

)
}
client?.clientType?.let {
MifosClientDetailsText(
Copy link
Contributor

Choose a reason for hiding this comment

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

remove the comments

Copy link
Author

Choose a reason for hiding this comment

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

Ok Sir

@amanna13
Copy link
Contributor

@SnehaghoshBarsha444 Thank you for our initial contribution in this issue. As the reviews were not addressed from a long time, hence a new PR #2578 has been opened with the related changes
Thanks !

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.

5 participants