Skip to content

Conversation

leightkt
Copy link
Contributor

Description

Update favicon on docs site

Screenshot 2023-03-28 at 12 04 33 PM

@leightkt leightkt self-assigned this Mar 28, 2023
@vercel
Copy link

vercel bot commented Mar 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
web3-onboard-docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 28, 2023 at 8:51PM (UTC)
@Adamj1232
Copy link
Contributor

This does seem to fix the issue but I feel we are missing the sveltekit way for loading our favicon. The fact that we are getting a 404 on page load makes me think we are missing a config for static files. Maybe something here?
If we choose this route should we remove the favicon within the project? cc @gesquinca

@leightkt
Copy link
Contributor Author

@Adamj1232 I was able to get it to load the existing favicon file by changing the path to /favicon.png

Copy link
Contributor

@Adamj1232 Adamj1232 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@leightkt leightkt merged commit 10b07be into develop Mar 29, 2023
@leightkt leightkt deleted the favicon branch March 29, 2023 15:48
Adamj1232 added a commit that referenced this pull request Apr 4, 2023
* Yarn docs * update favicon on docs site (#1611) * Update Notify section on the docs site and within readme (#1612) * Update notify docs on docs site and within readme * Update notify api docs * Add screenshots * Update signup link * Remove export from types in docs * Update Docs Favicon to Url (#1618) * Fix broken link in docs (#1616) * Add Kat to GH issue and notion card generation + Fix doc prop name for connect property (#1593) * Yarn the docs * Fix doc prop name for connect property * Add new notion Flag * Add Kat to gh issue notion cards * Update Broken Links on Docs Pages (#1619) * Update WalletConnect `eth_RequestAccounts` flow along with update WC deps and add new optional params (#1622) * Update WC packages * Add logs for testing * Refactor eth_requestAccounts flow * Why Don't I See My Wallet Link (#1617) * Add peer dep required to run package outside of walletconnect eco (#1623) * Update versions for release * Bump versions --------- Co-authored-by: Kat Leight <33187102+leightkt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants