safe-react-apps WalletConnect V2 icon is broken

The Safe app sets the WalletConnect V2 metadata icon to a relative URL "/images/safe-logo-green.png" while it should be a full URL including host. This causes the connected wallet to display a broken icon:

This happens when connecting a wallet to the Safe app using the WalletConnect option.

Jul 13 '23 09:07 jfschwarz

Thanks Jan!

@dasanra @JagoFigueroa could you guys check?

Jul 13 '23 09:07 katspaugh

Hi there! @jfschwarz could you share the dapp that you connected to via the WC safe app and it is using the icon from our app? I've tried with a few but I see that they usually use their own icon 😢

Cheers!

Jul 13 '23 11:07 JagoFigueroa

Hi @JagoFigueroa. Try with https://mech-omega.vercel.app/mechs/gno:0xb0dd814A7721b3fd2bA79f25dbA1f348383f8a31/679

(first connect any metamask account in the mech app, then there will be an input field for pasting the walletconnect key from Safe)

Jul 13 '23 12:07 jfschwarz

Thank you, I was able to reproduce it now 😄

No action from us needed here I would say señor @katspaugh as the connection is stablished from the web ui

https://github.com/safe-global/safe-react-apps/assets/92332710/6c3f5b1e-5012-4a1d-a56c-9e93993c1055

Cheers!

Jul 13 '23 12:07 JagoFigueroa