Fix: TxBuilder - pass txRecipient

[katspaugh]

What it solves

Resolves #3385

How this PR fixes it

The recipient field wasn't passed to the estimation endpoint in the Safe App Tx Review modal.

How to test it

• Create a tx with the Tx Builder
• Estimation should work

Note

A cool trick learned from @DiogoSoaress: you can filter the network request by method like so:

-method:OPTIONS estimations

Minus means exclude.

[github-actions]

CLA Assistant Lite All Contributors have signed the CLA.

[DiogoSoaress]

👍

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[github-actions]

Deployment links

🟠 Rinkeby	⚪ Mainnet	🟣 Polygon	🟡 BSC	⚫ Arbitrum	🟢 Gnosis Chain

[github-actions]

E2E Tests Failed
Check the results here: https://github.com/gnosis/safe-react-e2e-tests/actions/runs/1778593577

Failed tests:

• ❌ Safe Apps List Safe Apps List
• ❌ Read-only transaction creation and review Read-only transaction creation and review

[francovenica]

It works

The tx builder example that was given where it failed is not failing anymore
Also trying to upgrade the safe is not showing the error message anymore

[coveralls]

Pull Request Test Coverage Report for Build 1778553612

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 33.65%

---

Totals
Change from base Build 1777873195:	0.0%
Covered Lines:	3167
Relevant Lines:	8449

---

💛 - Coveralls