Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update favicon on docs site #1611

Merged
merged 2 commits into from
Mar 29, 2023
Merged

update favicon on docs site #1611

merged 2 commits into from
Mar 29, 2023

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
Member

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
Member

@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants