Only show SendTransaction on your accounts

[lukaw3d]

Before:

• user imports oasis1qqug..hsu8 to wallet
• search oasis1qr9t..gxfl
• see SendTransaction component to transfer from oasis1qqug..hsu8 to any Recipient
  

[github-actions]

MegaLinter status: ✅ SUCCESS

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ EDITORCONFIG	editorconfig-checker	3		0	0.01s
✅ GIT	git_diff	yes		no	0.01s
✅ TSX	eslint	3	0	0	4.61s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

[codecov]

Codecov Report

Merging #865 (5341f2e) into master (7d07e97) will decrease coverage by 0.02%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #865      +/-   ##
==========================================
- Coverage   88.05%   88.02%   -0.03%     
==========================================
  Files          96       96              
  Lines        1599     1604       +5     
  Branches      334      336       +2     
==========================================
+ Hits         1408     1412       +4     
- Misses        191      192       +1     

Flag	Coverage Δ
cypress	58.16% <85.71%> (+0.06%)	⬆️
jest	74.57% <28.57%> (-0.18%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...ges/AccountPage/Features/SendTransaction/index.tsx	92.59% <66.66%> (-3.41%)	⬇️
...ages/AccountPage/Features/AccountDetails/index.tsx	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d07e97...5341f2e. Read the comment docs.

[buberdds]

I get idea that entire app will blow up to catch an error, but I think it's better to cover such scenarios in unit tests than in a component via new Error.

[lukaw3d]

I don't think tests can ensure that we devs don't move SendTransaction outside the condition:

      {isAddressInWallet && (
        ...
          <SendTransaction

Maybe with integration tests of all possible user flows 🤷