MetaMask / MetaMask/metamask-mobile

[Bug] Eth Sign transaction is unintentionally rejected after going back from `Learn more` link

Open
#6,409 2 comments 0 reactions 0 assignees View on GitHub
area-signatures Sev3-low 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 are performing an `eth_sign` we see that there is some warning information with a link. If we click the Learn more link, another page is open with some information, but once this is done, if you go back, the signature will be automatically rejected. There is no way to go back to the signature without being rejected automatically.

**Screenshots**

https://github.com/MetaMask/metamask-mobile/assets/54408225/add9c28a-9715-4c61-b2de-cfef6acc11af

**To Reproduce**
_Steps to reproduce the behavior_
1. Go to test dapp
2. Click on eth sign
3. Click on Learn more
4. Click on Back arrow
5. See how signature is rejected unintentionally

**Expected behavior**
We should be able to go back to the Signature screen. cc @bschorchit

**Smartphone (please complete the following information):**
- Device: Pixel 6
- OS: Android
- App Version: `v6.6.0` and possibly earlier

-------------------------------------------------------------
_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

Reproduce the issue in the test dapp on Android by starting an eth_sign request, opening Learn more, and using the back arrow. Trace the signature and Learn more navigation flow; done means the back action returns to the Signature screen without automatically rejecting the request.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.