Skip to content

fix: resolve menu rendering, syntax errors, and duplicate UI bugs#496

Open
PARNITA-SINGH wants to merge 1 commit into
PatelHarsh2006:mainfrom
PARNITA-SINGH:main
Open

fix: resolve menu rendering, syntax errors, and duplicate UI bugs#496
PARNITA-SINGH wants to merge 1 commit into
PatelHarsh2006:mainfrom
PARNITA-SINGH:main

Conversation

@PARNITA-SINGH

@PARNITA-SINGH PARNITA-SINGH commented Jun 5, 2026

Copy link
Copy Markdown

🔗 Related Issue
Closes #494

What was fixed

  • Added missing <script src="js/recommendations.js"> to menu.html
  • Fixed duplicate removedItem declaration in removeFromCart()
  • Removed require() statements incompatible with browser environment
  • Removed export keyword from functions loaded as plain scripts
  • Fixed stray ); causing SyntaxError in event listener
  • Removed orphaned newFeatures and main text artifacts from merge
  • Removed duplicate Add button and broken ₹undefined / NaN% OFF UI
  • Removed cartManager.validate() call for undefined method

Before & After
Menu items now render correctly with clean price display and single Add button.

BEFORE
image

AFTER
image

@PARNITA-SINGH

PARNITA-SINGH commented Jun 6, 2026

Copy link
Copy Markdown
Author

Hi @PatelHarsh2006 , Just following up on this PR — I'd love to get some feedback. This was submitted as part of GSSoC'26 and also add the label of GSSOC to my PR. Please let me know if any changes are needed! 🙂

@PARNITA-SINGH

Copy link
Copy Markdown
Author

Hi @PatelHarsh2006 ,

The requested changes have been completed and pushed. I also checked the Vercel status, and the deployment is currently awaiting authorization rather than failing due to a build error.

Could you please review the PR when convenient? If everything looks good, I'd appreciate it if the PR could be labeled for GSSoC'26 and approved/merged.

Thank you!

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.

Bug: Menu Items Not Rendering from JSON Data

1 participant