Closed aalavandhan closed 3 years ago
Nice work - this all looks great and makes sense. It's way nicer to just have the balances float on query instead of explicit rebase like my original idea.
I assume writing tests / hooking the existing Fragments tests into this implementation is future scoped work? I have a few concerns about precision loss that may occur, would like to test and verify
Yea. I'll get to the unit testing in a future PR
Unrelated question and non-blocking - do you have a gpg key w/ which you can sign commits? Generally good opsec habit I'd like our team to follow
Basic rebasing button token wrapper