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

feat(feature:account): set up initial directory structure for account… #2773

Merged

Conversation

HekmatullahAmin
Copy link
Contributor

… modules

  • Created directory structures for account, savings-account, loan-account, and share-account modules.
  • Added initial build.gradle.kts files and configured dependencies for each module.
  • Prepared account module to include other modules as dependencies for future integration.

Fixes - Jira-#186

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.

… modules

- Created directory structures for `account`, `savings-account`, `loan-account`, and `share-account` modules.
- Added initial `build.gradle.kts` files and configured dependencies for each module.
- Prepared `account` module to include other modules as dependencies for future integration.
@HekmatullahAmin
Copy link
Contributor Author

@niyajali this is how the structure looks like. If you approve then kindly merge so I work on other modules.

Screenshot 2025-02-16 at 13 35 30

@niyajali
Copy link
Collaborator

niyajali commented Feb 16, 2025

@HekmatullahAmin I don't think this module contains any viewModels, no issues merging this for now we'll refactor later

@niyajali niyajali enabled auto-merge (squash) February 16, 2025 13:50
@niyajali
Copy link
Collaborator

@HekmatullahAmin go ahead work on other modules, this PR will be merged soon

@HekmatullahAmin
Copy link
Contributor Author

@HekmatullahAmin I don't think this module contains any viewModels, no issues merging this for now we'll refactor later

maybe I just added like that if i see nothings is needed i will delete it

@niyajali niyajali merged commit 9bec46e into openMF:kmp-impl Feb 16, 2025
7 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.

2 participants