Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Signing onchain transactions in mobile MM/1Inch goes through opening the app in AppStore #510

Closed
elena-zh opened this issue May 4, 2022 · 9 comments
Labels
app:CowSwap CowSwap app Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@elena-zh
Copy link

elena-zh commented May 4, 2022

Related to #502

Originally reported in #444 (comment)

Some more context: #444 (review)

I see a request to open App store when I send a request to Approve a token using MM
image

However, I proceeded and opened the AppStore, then opened MM from there, and was able to sign the approve transactions. Afterwards I was redirected back to the app store.
The same is for the warp/unwrap/convert transactions.
Here is the video: https://drive.google.com/file/d/1jR5AuVWL0O-PMRuIPdQod5uc7AGR-Hm4/view?usp=sharing

The same is for the 1Inch wallet. However, the CowSwap app page is changed to that one https://wallet.1inch.io/wc/, and transaction does not appear in the activity modal when go back to the Cowswap app.

@elena-zh elena-zh added app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional labels May 4, 2022
@fairlighteth
Copy link
Contributor

Hehe, nice music selection 🎶 !

So following the screen recording, it looks like:

  • It brings you to the AppStore even though you already had MetaMask installed?
  • When you click on OPEN in the AppStore, it opens your already installed MetaMask and you see the transaction signing request. When you sign, nothing seems to be signed/transmitted to the UI?

@elena-zh
Copy link
Author

elena-zh commented May 5, 2022

@fairlighteth , thanks=)

  1. Yes, it opens AppStore even the app is already installed
  2. When I sigh:
  • for MM: the Tx is signed, but I'm navigated back to the AppStore, and need 'manually' go back to the browser to open the Cowswap app
  • for 1Inch: the TX is signed, but nothing is transmitted to the UI

@elena-zh
Copy link
Author

elena-zh commented May 5, 2022

I have retested the issue in #502 PR and I faced this intermediate message only once after connection a wallet in MM when testing in iOS. Then transactions started to run nicely with MM.
Video: https://cowservices.slack.com/files/UHCSH5J5C/F03E7303PR8/mm.mp4

As for the 1Inch wallet, I always face an intermediate step when try to sign onchain transactions in iOS. However, transactions started to be visible in the Cowswap.

in Android all types of transactions work fine for both 1Inch and MM wallets

@elena-zh
Copy link
Author

elena-zh commented May 5, 2022

Still reproducible in iOS in Staging.
I do not know why, but sometimes I do not see an intermediate step, sometimes - always see it (MM connection)

@anxolin
Copy link
Contributor

anxolin commented May 5, 2022

Apparently is a well know bug https://community.metamask.io/t/deeplink-opens-appstore-when-app-installed/18199/20 a bit sad

@anxolin
Copy link
Contributor

anxolin commented May 5, 2022

Also a related issue, MetaMask/metamask-mobile#3965

They show there a workaround (MetaMask/metamask-mobile#3965 (comment)), but we are not the ones generating the deep link (is Wallet connect), and second, there's someone that says that it doesn't work

@anxolin
Copy link
Contributor

anxolin commented May 5, 2022

Also related issue, here they mentioned that is solved in MM 5.0.1, but for me it still reproduces.
MetaMask/metamask-mobile#3914

Very annoying also that Uniswap uses web-react and they don't have this issue afait.

@anxolin
Copy link
Contributor

anxolin commented Jul 5, 2022

Theres a few solutions suggested here MetaMask/metamask-mobile#3965 @nenadV91 can you take a look?

@elena-zh
Copy link
Author

elena-zh commented Apr 5, 2023

@elena-zh elena-zh closed this as completed Apr 5, 2023
@elena-zh elena-zh reopened this Jun 26, 2023
@elena-zh elena-zh added Low Severity indicator for defects. It won't cause any major break-down of the system and removed Medium Severity indicator. It causes some undesirable behavior, but the system is still functional labels Jun 26, 2023
@elena-zh elena-zh closed this as completed Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:CowSwap CowSwap app Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Development

No branches or pull requests

4 participants