search

hemilabs / ui-monorepo

UI monorepo for Hemi Labs
https://app.hemi.xyz/
MIT License
2 stars 1 forks source link

Cannot read properties of undefined (reading 'address') #637

Open gndelia opened 3 weeks ago

gndelia commented 3 weeks ago

Description

It seems somehow the app is trying to instantiate a cross chain messenger instance with an L1 address that is not configured. This causes the error mentioned above. See Sentry report

Steps to reproduce

TBD

Environment

Proposed fix

sentry-io[bot] commented 3 weeks ago

Sentry Issue: PORTAL-8

gndelia commented 3 weeks ago

This error (as in app-crashing and reporting to Sentry) shouldn' happen anymore after #648 However, although there won't be an error anymore, the app "hangs" in a loading state. See

Image

This still needs to be fixed. A possible scenario to reproduce this is when trying to access an operation that the user doesn't have access. For example:

  1. Connect one account, and perform a withdraw.
  2. Wait until the "Prove" operation becomes available
  3. Open the "Prove" operation and bookmark the url. DO NOT PROVE THE WITHDRAW
  4. Now disconnect the account, and try connecting another account. (Or use the disconnected mode)
  5. Click the bookmarked url. The operation is not saved in the local storage, and can't be loaded, so nothing happens (And hangs in a "Loading" mode

I don't this necessarily requires another account. I think (I haven't reproduce it yet) this can be reproduce with one account as well:

  1. Pick an old withdrawal that hasn't been proven or claimed
  2. Open the "Claim|Prove" operation and bookmark the url. DO NOT CLAIM|PROVE THE WITHDRAW
  3. Reset the transaction history
  4. Before the unclaimed/unproven withdrawal becomes visible, open the bookmarked URL
  5. Same should happen - app hangs