-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Adding Favicon to the webpage. #296
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you even test your PR before opening it? Because if you had you would have noticed the favicon is not being displayed 🤔
It's just a simple path issue, so we should be able to merge once that's fixed (see details in comments).
As i have said before cannot build with webpack. I test it locally on vite haha. |
Ah I see, it explains ^^ Just so you know you can just run Then when you check out your vite branch, you just run |
I tried but webpack gives errors |
What kind of Webpack errors @nakul-py ? Because reinstalling the packages when changing branch does work. If you are using |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build CI does not work. Probably the path to the favicon is incorrect relative to the html file.
Ok i will do this later when project runs with vite. |
No problem, we are not in a hurry 👍 |
Description
Adding Favicon to the Cosmos Journeyer Webpage.
Fixes #293