MetaMask / MetaMask/metamask-mobile
Display on-chain failure reason on tx confirmation
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
Mobile version of https://github.com/MetaMask/metamask-extension/issues/9419
We currently show a warning when a transaction is likely to run out of gas, but we could also show the error text related to any transaction failure.
This probably wouldn’t require any additional design, and could simply be added into the current warning field.
https://twitter.com/smpalladino/status/1305991209257775105?s=12
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the linked MetaMask extension issue 9419 and locating the existing transaction warning field in the mobile confirmation flow. The work is done when the confirmation displays the error text for failed on-chain transactions, alongside the existing likely-out-of-gas warning.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- blockchain, mobile
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100