MetaMask / metamask-mobile

Mobile web browser providing access to websites that use the Ethereum blockchain
https://metamask.io
Other
2.15k stars 1.11k forks source link

The app is throwing a non descriptive error when opening our staging app URL : View: Root Error: Text strings #2716

Closed LastKenobi closed 3 years ago

LastKenobi commented 3 years ago

Describe the bug We started testing our new environment for AirNFTs and when we tested on Metamask Mobile it suddenly started failing when opening the URL https://app.airnfts.com/ It logged the error below 👇

View: Root
Error: Text strings must be rendered within a <Text> component.

I checked on older issues on this repo, it seems to have come back many times, the issue here is we can't debug it since it's non descriptive, we can't know know for sure which strings threw that error.

Just for more contextual info, this URL doesn't crash on Metamask chrome plugin, only the mobile app.

Screenshots

photo_2021-05-23 02 42 28

To Reproduce Steps to reproduce the behavior

  1. Go to browser
  2. Type on site url
  3. See error

Expected behavior Metamask shouldn't crash on opening a dapp URL, and if it would crash it should provide a precise error message that would help on debugging

Smartphone (please complete the following information):


to be added after bug submission by internal support / PM Severity

oesukam commented 3 years ago

Hi @LastKenobi , I'm facing the same issue. Did you manage to resolve the issue?

I have just raised a PR, hope to be approved soon. PR

oesukam commented 3 years ago

@LastKenobi You can correct your website's favicon.ico as a quick fix to resolve your issue

LastKenobi commented 3 years ago

Issue was fixed Thanks @oesukam