fix: fix contract address display in confirmation page

[sahar-fehri]

Description

Sometimes the confirmation transaction page displays the destination wallet address rather than the contract address.

Here is related github issue: #19502.

Manual testing steps

_1. Go to : https://metamask.github.io/test-dapp/ or you can use etherscan on a contract address
_2. Deploy ERC1155 smart contract
_3. Call safeBatchTransferFrom function and notice the confirmation page does displays the destination address and not the contract address (side by side with the contract function name)

Screenshots/Recordings

Before

Calling mintBatch fct displays contract address correctly

Calling safeBatchTransferFrom fct displays the destination address instead:

After

Mint batch and safeTransferFromBatch display contract address

And if you are sending a transfer transaction (contractInteraction) to an address already imported in your wallet, it shows :

Related issues

Fixes #MMAssets-33

Pre-merge author checklist

• [x ] I’ve followed MetaMask Coding Standards.
• [ x] I've clearly explained:
  • What problem this PR is solving.
  • How this problem was solved.
  • How reviewers can test my changes.
• [x ] I’ve indicated what issue this PR is linked to: Fixes #???
• I’ve included tests if applicable.
• [ x] I’ve documented any added code.
• [ x] I’ve applied the right labels on the PR (see labeling guidelines).
• I’ve properly set the pull request status:
  • In case it's not yet "ready for review", I've set it to "draft".
  • In case it's "ready for review", I've changed it from "draft" to "non-draft".

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[metamaskbot]

Builds ready [b55a326]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4
  • common: 0, 1, 2, 3, 4, 5, 6
  • content-script: 0
  • contentscript: contentscript
  • disable-console: disable-console
  • inpage: inpage
  • policy-load: policy-load
  • sentry-install: sentry-install
  • ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (657 ± 335 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	81	123	94	12	6
		domContentLoaded	65	122	86	17	8
		load	76	1631	657	698	335
		domInteractive	65	122	86	17	8

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 26 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (0a215ca) 68.43% compared to head (b52ccad) 68.43%.
Report is 2 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #21042   +/-   ##
========================================
  Coverage    68.43%   68.43%           
========================================
  Files         1012     1012           
  Lines        40492    40495    +3     
  Branches     10825    10826    +1     
========================================
+ Hits         27707    27711    +4     
+ Misses       12785    12784    -1     

Files	Coverage Δ
...ummary/confirm-page-container-summary.component.js	87.80% <100.00%> (+3.59%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [b52ccad]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4
  • common: 0, 1, 2, 3, 4, 5, 6, 7
  • content-script: 0
  • contentscript: contentscript
  • disable-console: disable-console
  • inpage: inpage
  • policy-load: policy-load
  • sentry-install: sentry-install
  • ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (713 ± 361 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	89	143	102	14	7
		domContentLoaded	71	157	94	20	10
		load	81	1864	713	752	361
		domInteractive	71	157	94	20	10

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 26 Bytes (0.00%)
• common: 0 Bytes (0.00%)