search

ephemeraHQ / xmtp-inbox-web

XMTP Inbox web chat app
https://dev.xmtp.chat
MIT License
33 stars 40 forks source link

Share QR Code (DM) only works sometimes... (Critical Bug) #351

Closed airoasis closed 1 year ago

airoasis commented 1 year ago

Describe the bug Actually, dm (share qr code) only works sometimes... Most of time, it doesn't work.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'https://dev.xmtp.chat/dm/0x095a51D628BED9a8e9a9f2E9e340E94e21EBe13A' (you can try any address that is connected to xmtp dev nodes)
  2. Repeat it again and again It only works sometimes...

Expected behavior The XMTP Inbox app should display a new message window with a specific eth address.

Screenshots Most of time it only shows the empty address

image

In the mobile view, it shows the conversation list when the DM doesn't work

image

Even when the DM works, it doesn't display the eth address in the mobile view.

image

Device and browser: Macbook Pro Ventura Chrome browser

airoasis commented 1 year ago

If I put the following code right after the line https://github.com/xmtp-labs/xmtp-inbox-web/blob/d7475dfe72d64d5f058781b0abd526e9f8e80c93/src/pages/dm.tsx#L27, it fixes the issue in the desktop view, but not mobile view. The variable routeToInbox is async, but setRecipientWalletAddress is not, therefore it only works sometimes...

await new Promise(resolve => setTimeout(resolve, 1000));

However, in the mobile view, wallet adress resolving is not working even after this.

rygine commented 1 year ago

@airoasis we've merged in a fix for this to https://xmtp.chat. please let us know if you're still experiencing it.

airoasis commented 1 year ago

@rygine Unfortunately, still it has the same issue... It only works sometimes....