-
Notifications
You must be signed in to change notification settings - Fork 86
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
added the fav icon #41 #68
Conversation
✅ Deploy Preview for delightful-daifuku-a9f6ea 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.
🎉 Thank you @Kratik1093 for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.
@Kratik1093 If it's visible at your side can you share the image. |
frontend/package.json
Outdated
"main": "postcss.config.js", | ||
"author": "", | ||
"license": "ISC", | ||
"description": "" |
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.
@Kratik1093 why did you added these lines here?
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.
to make this page run on my device i need to made these change otherwise the page was not starting on my device
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.
@Kratik1093 then please remove this while pushing the PR and use it for local only
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.
ok will keep note of it for next time
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.
@Kratik1093 the above lines need to be removed to merge this Pr. Please remove it asap.
Please reply to the comment and I am still not able to see the icon. |
@AlfiyaSiddique I am also could understand why this is not working when deployed i checked the deployed code also but iant able to understand the problem is the image in public section named tastytrial512.png , changes in html are visible to you in the deploy?? |
@Kratik1093 please update the pr and add the chnages I have commented |
@Kratik1093 any updates? |
@Kratik1093 will be closing this PR and issue if no activity found by tomorrow. |
👷 Deploy request for delightful-daifuku-a9f6ea pending review.Visit the deploys page to approve it
|
added the new fav icon which matches theme of the food website
issue no. #41