Closed bergeron closed 1 week ago
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.
Thnx for the fix! 🙏
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 67.38%. Comparing base (
291a642
) to head (0195d92
). Report is 3 commits behind head on develop.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
Fixes an issue when you changed accounts while sending an NFT. It would keep the NFT asset, even though the new account doesn't own it. Reset to the native asset instead.
Related issues
Fixes: https://github.com/MetaMask/metamask-extension/issues/24391
Manual testing steps
Screenshots/Recordings
Before
https://github.com/MetaMask/metamask-extension/assets/3500406/7628c507-a694-4bf9-8d1b-e828f7a8cdc7
After
https://github.com/MetaMask/metamask-extension/assets/3500406/1ac60586-cf97-47bc-9f4e-1e518b8a053b
Pre-merge author checklist
Pre-merge reviewer checklist