Remove wrapper fee unshielding
Describe your changes
Partially addresses #2597.
Removes the optional fee unshielding Transaction object from WrapperTx. Removes/refactors associated functions, types and tests. Removes fee unshielding support from the client and the sdk.
NOTE: with this PR only transactions that pay fees from the signer balance are accepted. If you need to test masp transactions you'll need to have enough funds in a transparent balance.
Indicate on which release or other PRs this topic is based on
#3103 (diffs for review: https://github.com/anoma/namada/pull/3217/files/7f4b6c1e6656ec44194562ad1e167b09d24170f4..cf52c605e33279f4f464b210a8e215a7138be809)
Checklist before merging to draft
- [x] I have added a changelog
- [x] Git history is in acceptable state
Codecov Report
Attention: Patch coverage is 46.82689% with 1416 lines in your changes are missing coverage. Please review.
Project coverage is 60.63%. Comparing base (
4ed6229) to head (c2e98ab).
Additional details and impacted files
@@ Coverage Diff @@
## main #3217 +/- ##
==========================================
+ Coverage 60.24% 60.63% +0.38%
==========================================
Files 303 303
Lines 93191 93550 +359
==========================================
+ Hits 56145 56721 +576
+ Misses 37046 36829 -217
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@yito88 this pr changes the structure of the Tx once more so we'll probably need another update of Hermes