Skip to content

Fix frontend image paths,add gitignore, update category state, README…#4

Open
prem2621 wants to merge 1 commit into
Mshandev:mainfrom
prem2621:fix
Open

Fix frontend image paths,add gitignore, update category state, README…#4
prem2621 wants to merge 1 commit into
Mshandev:mainfrom
prem2621:fix

Conversation

@prem2621

Copy link
Copy Markdown

… instructions and updates

🚀 Summary

This pull request includes key frontend fixes,cart Update, Readme improvements, and documentation enhancements.


🔧 Changes Made

🎨 Frontend Fix

  • Corrected food images not displaying by adjusting image source paths in FoodItem.jsx.

🛒 ExploreMoreMenu Fix

  • Fixed error prev=>prev===item.menu_name?"All":item.menu_name) instead of prev=>prev===item.menu_image.name?"All":item.menu_name If you use item.menu_image.name it doesn't update the state because the category name item.menu_name so I just modify that

📘 README Update

  • Added note for contributors to use their own .env file.

  • Enhanced formatting with a more colorful and structured README for better readability and contributor engagement.

  • adding live deployment links for Frontend, Backend, and Admin panels — it helps viewers and contributors quickly test the project.

💡 Why These Changes Matter

  • ✅ Improves frontend UX by fixing image rendering and button cursor in css.

  • 🛡️ Prevents app crashes during empty cart states.

  • 🔒 Add deployment instruction step by step so make sure user can understand

  • 📚 Makes documentation more readable and contributor-friendly.

@codeCraft-Ritik codeCraft-Ritik left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! The implementation is clean and easy to understand

@Greetlinj

Copy link
Copy Markdown

Good idea to enhance this project...

@trungng29 trungng29 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

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.

4 participants