ivy-wallet
ivy-wallet copied to clipboard
[Domain] Build the `AccountBalanceUseCase`
Please confirm the following
- [X] I checked the current issues for duplicate issues.
What would you like to improve?
- Build the
AccountBalanceUseCase. - Use the
AccountStatsUseCaseand theTransactionRepository
class AccountBalanceUseCase @Inject constructor(...) {
fun calculate(account: AccountId): Map<AssetCode, NonZeroDouble>
}
- Build the
NonZeroDoublefollowing the pattern to thePositiveDouble - We neee NonZeroDouble to handle the case of negative balances (credits to @Vishwa-Raghavendra)
Because...
- Proper domain logic following the architecture and having unit + property-based tests.
Description
No response
Success Criteria
- Follows all dev guidelines (this is a critical code so expect Requests for changes on your PR)
- Property-based tests
- Unit tests
Thank you @ILIYANGERMANOV for raising Issue #3143! 🚀 What's next? Read our Contribution Guidelines 📚.
Tagging @ILIYANGERMANOV for review & approval 👀
I'm on it
"I'm on it"
Thank you for your interest @priyanshuborole! 🎉 Issue #3143 is assigned to you. You can work on it! ✅
If you don't want to work on it now, please un-assign yourself so other contributors can take it.
Also, make sure to read our Contribution Guidelines.
I'm on it
Thank you for your interest @rishi2062! 🎉 Issue #3143 is assigned to you. You can work on it! ✅
If you don't want to work on it now, please un-assign yourself so other contributors can take it.
Also, make sure to read our Contribution Guidelines.
Hey @rishi2062 any updates on this? If not you can unassign yourself so someone else can take it
Sure, although i started but not getting time lately.🥲
@ILIYANGERMANOV can you help me with the unassign code.
I'm on it
Thank you for your interest @rodrigoliveirac! 🎉 Issue #3143 is assigned to you. You can work on it! ✅
If you don't want to work on it now, please un-assign yourself so other contributors can take it.
Also, make sure to read our Contribution Guidelines.