-
Notifications
You must be signed in to change notification settings - Fork 62
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
Feat/favicon missing #39
Feat/favicon missing #39
Conversation
✅ Deploy Preview for dsr-isrowebapp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Great job, @pankaj-2503! 🎉 Thank you for submitting your first pull request. Your contribution is valuable and I appreciate your efforts to improve my project.
I will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check my contributing guidelines
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.
After reviewing the deployed view, I've noticed that there is no favicon visible.
I have made changes and pushed a new PR you can review it and merge
afterwards
…On Sun 8 Oct, 2023, 10:10 PM Daksh Singh Rathore, ***@***.***> wrote:
***@***.**** requested changes on this pull request.
After reviewing the deployed view, I've noticed that there is no favicon
visible.
—
Reply to this email directly, view it on GitHub
<#39 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVO5VAG3DBPH6PT2HRTNNTDX6LJQVAVCNFSM6AAAAAA5XWT7EKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNRTGM2DKMZZGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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.
Attach the screenshot of the deployed website, with favicon showing as it should be!
@@ -4,6 +4,7 @@ | |||
<meta charset="UTF-8" /> | |||
<meta name="viewport" content="width=device-width, initial-scale=1.0" /> | |||
<link rel="stylesheet" href="css/style.css" /> |
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.
It is showing in website why you are not able to see it i don't understand it
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.
lgtm !!
What this PR introduces
What is the need of this PR
Fixes #32