Bug: Unable to define appLogoUrl

[homeyong]

Describe the bug

Unable to define appLogoUrl for coinbase browser extension

Steps

1. Go to https://codesandbox.io/s/web3modal-demo-forked-obbuf5?file=/src/providerOptions.js.
2. Define appLogoUrl, in my example it is twitter.ico
3. Refresh the sandbox browser
4. Click on the Connect > Select Coinbase > noticed that the icon does not change to twitter.
   

Expected behavior

The Icon should in the coinbase extension show the logo that defined in appLogoUrl.

Version

3.0.1

Additional info

No response

Desktop

No response

Smartphone

No response

[pchy10]

Any progress on this? I am also getting a similar kind of error.

Currently, I am seeing a broken image while it Is showing the user permission window. I am using the following code to initialize the CoinbaseWalletSDK:

import ethers from "ethers";
import CoinbaseWalletSDK from "@coinbase/wallet-sdk";

const CHAIN_ID = 80001;
// const RPC_URL = ...
// ...
const coinbaseWallet = new CoinbaseWalletSDK({
  appName: "Affine",
  appLogoUrl: "/favicon.io"
});

const ethereum = coinbaseWallet.makeWeb3Provider(RPC_URL, parseInt(CHAIN_ID, 16));

const provider = new ethers.providers.Web3Provider(ethereum);
// ...

But after the connection has been made, the Dapp Connections on the extension is showing the title & favicon of that route:

Update

The version I am using is ^3.5.3 for @coinbase/wallet-sdk and ^5.6.5 for ethers.

[immunity20]

Any update for fixing the empty image?
@petejkim You need to update your Coinbase Extension.

You try to display website's favicon.ico but instead of <img src="favicon_url" /> you do <img alt="favicon_url" />

[gareys-cb]

Hey! Thanks for reaching out.

If you are still working on this, please try the following:

1. Ensure that your appLogoUrl is a complete URL, not a relative path. i.e. https://mysite.com/logo.png
2. Provide a browser supported image filetype such as .png or .jpg. See the supported filetypes.

We will update our docs to reflect these requirements. Sorry for the confusion.

[pchy10]

Hey! Thanks for reaching out.

If you are still working on this, please try the following:

1. Ensure that your appLogoUrl is a complete URL, not a relative path. i.e. https://mysite.com/logo.png
2. Provide a browser supported image filetype such as .png or .jpg. See the supported filetypes.

We will update our docs to reflect these requirements. Sorry for the confusion.

@gareys-cb I used an external URL as mentioned, but still seeing the same issue. For reference, I am using this URL as appLogoUrl value - https://cdn.affinedefi.com/affine/logo-512x512.png

[Hlllllllii]

🇲🇽Sent from my iPhoneOn Feb 6, 2023, at 9:06 AM, Prosenjit Chowdhury ***@***.***> wrote: Hey! Thanks for reaching out. If you are still working on this, please try the following: Ensure that your appLogoUrl is a complete URL, not a relative path. i.e. https://mysite.com/logo.png Provide a browser supported image filetype such as .png or .jpg. See the supported filetypes. We will update our docs to reflect these requirements. Sorry for the confusion. @gareys-cb I used an external URL as mentioned, but still seeing the same issue. For reference, I am using this URL as appLogoUrl value - https://cdn.affinedefi.com/affine/logo-512x512.png —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>