MetaMask / MetaMask/metamask-mobile

[Bug] Hex data is not displayed when the option is enabled for Contract Deployment or Contract Interaction transactions

Open
#6,773 1 comment 0 reactions 0 assignees View on GitHub
Sev2-normal team-confirmations-planning type-bug type-pinned
Dominant language
TypeScript
Stars
3k
Forks
1.7k
Avg merge
1d 14h
Merged PRs (30d)
669

Description

**Describe the bug**
Whenever we enable the Hex Data option from Advanced Settings, we should see the Hex Data on any transaction. However this does not happen for Contract Deployments or non custom Contract Interactions

**Screenshots**

https://github.com/MetaMask/metamask-mobile/assets/54408225/8050703f-f535-4103-b0aa-d26857f91d78

**To Reproduce**
_Steps to reproduce the behavior_
1. Go to Advanced Settings
2. Enable Hex Data
3. Go to test dapp
4. Click Deploy NFT (this will trigger a Contract Deployment transaction)
5. See no Hex Data is displayed
6. Click Mint (this will trigger a non-custom Contract Interaction transaction)
7. See no Hex Data is displayed

**Expected behavior**
We should see the Hex Data for Contract Deployments/Interactions if the option is enabled on Advanced settings. cc @bschorchit on Extension we have a tab, I think on Mobile the design is different? :

![Screenshot from 2023-07-10 12-47-40](https://github.com/MetaMask/metamask-mobile/assets/54408225/b7fe3d1c-9770-4fda-8757-85d0920c6c24)

**Smartphone (please complete the following information):**
- Device: Pixel 6
- OS: Android
- App Version: `7.2.0` possibly in earlier versions too

-------------------------------------------------------------
_to be added after bug submission by internal support / PM_
**Severity**

- How critical is the impact of this bug on a user?
- Add stats if available on % of customers impacted
- Is this visible to all users?
- Is this tech debt?

Contributor guide

Open the contributing guide

Research direction

Start in the test dapp by enabling Hex Data from Advanced Settings, then reproduce the Deploy NFT and Mint transactions. Trace the mobile transaction display for Contract Deployment and non-custom Contract Interaction transactions; done means Hex Data is visible for both when the option is enabled.

Written by the indexing model from the issue text.

Assessment

Tech stack
react-native, typescript
Domain
frontend, mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.