MetaMask / MetaMask/metamask-mobile
Increase friction to bypass estimated revert
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
Cross-posted as https://github.com/MetaMask/metamask-extension/issues/11966
[A twitter user comments](https://twitter.com/0xfoobar/status/1431805767003230209?s=21) that many new users are confused by having an estimated gas as well as a "transaction expected to fail" warning, because it implies that maybe this is what the transaction would cost to execute normally.
One possible improvement:
- If the transaction would revert/fail, hide the gas estimate info at first, as well as maybe disabling the confirm button.
- Need to still provide a way for pro users to override this. Something like "I want to submit this anyway", which would reveal the gas estimate and the confirm button.
This would need design, and then be a cross-product change.
Rating this as a sev2, but it's probably a high sev2, since it scares new users.
Contributor guide
Research direction
The issue names no files, tests, or entry points; begin by reviewing the mobile transaction confirmation flow and the linked cross-posted issue. Define the warning, gas estimate, and pro-user override behavior with design before implementation; done means the agreed cross-product experience is implemented and verified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- blockchain, mobile
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100