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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants