ivy-wallet icon indicating copy to clipboard operation
ivy-wallet copied to clipboard

Migrate `TransactionDao` calls to `TransactionRepository`

Open ILIYANGERMANOV opened this issue 10 months ago • 1 comments

Please confirm the following

What would you like to improve?

  • Migrate all TransactionDao calls to TransactionRepository (if the repo method is missing, you need to create it and add tests for it)
  • Migrate all WriteTransactionDao calls to `TransactionRepository

Because...

  • Use the new Data layer that's single source of truth.
  • Fixes bugs ans inconsistencies.

Description

No response

Success Criteria

  • You've tested each change manually.
  • The code is covered with tests.
  • The code is consistent with the other stuff in :shared:data.
  • The code is following the Ivy Wallet's Developer Guidelines

ILIYANGERMANOV avatar Apr 14 '24 10:04 ILIYANGERMANOV

Thank you @ILIYANGERMANOV for raising Issue #3132! 🚀 What's next? Read our Contribution Guidelines 📚.

Tagging @ILIYANGERMANOV for review & approval 👀

ivywallet avatar Apr 14 '24 10:04 ivywallet

I'm on it

rishi2062 avatar Jun 02 '24 08:06 rishi2062

Thank you for your interest @rishi2062! 🎉 Issue #3132 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.

ivywallet avatar Jun 02 '24 08:06 ivywallet