fix: add validation error on NFT import

[sahar-fehri]

Description

Adds an error msg if contract address is not validation on the NFT import modal.

(Just to make the behavior more consistent with the token-import modal)

Manual testing steps

_1. Click on "import nft" button in the NFT tab
_2. Add invalid contract address and notice error msg

Screenshots/Recordings

Before

After

Fixes

https://consensyssoftware.atlassian.net/browse/MMASSETS-35

Pre-merge author checklist

• [x ] I’ve followed MetaMask Coding Standards.
• I've clearly explained:
  • What problem this PR is solving.
  • [ x] How this problem was solved.
  • [ x] How reviewers can test my changes.
• [ x] I’ve indicated what issue this PR is linked to: Fixes #???
• I’ve included tests if applicable.
• I’ve documented any added code.
• [x ] I’ve applied the right labels on the PR (see labeling guidelines).
• [ x] 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 [5ab689d]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): 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 (1735 ± 80 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	86	149	108	18	9
		domContentLoaded	74	144	104	18	9
		load	1408	2088	1735	167	80
		domInteractive	74	144	104	18	9

Bundle size diffs

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

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (941b734) 68.35% compared to head (2ba0590) 68.39%.

❗ Current head 2ba0590 differs from pull request most recent head 4c85958. Consider uploading reports for the commit 4c85958 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #21028      +/-   ##
===========================================
+ Coverage    68.35%   68.39%   +0.05%     
===========================================
  Files         1007     1007              
  Lines        40260    40256       -4     
  Branches     10770    10765       -5     
===========================================
+ Hits         27516    27533      +17     
+ Misses       12744    12723      -21     

Files	Coverage Δ
.../multichain/import-nfts-modal/import-nfts-modal.js	95.08% <100.00%> (+0.35%)	⬆️

... and 2 files with indirect coverage changes

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

[NidhiKJha]

This looks really good @sahar-fehri , before I approve one follow-up question. Do we want to show the error message when the address field is blank, I think in this case we should not show any message.

[metamaskbot]

Builds ready [2ba0590]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): 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 (990 ± 347 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	84	151	107	16	8
		domContentLoaded	68	121	100	16	8
		load	79	1675	990	724	347
		domInteractive	68	121	100	16	8

Bundle size diffs

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

[metamaskbot]

Builds ready [504aac0]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): 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 (1044 ± 410 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	93	150	110	17	8
		domContentLoaded	74	158	101	23	11
		load	90	2118	1044	855	410
		domInteractive	74	158	101	23	11

Bundle size diffs

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

[NidhiKJha]

LGTM!