Skip to content

fix: total accumulation bug, HTML in JS file, orderBtn null crash - #953

Open
saurabhhhcodes wants to merge 1 commit into
Anjaliavv51:mainfrom
saurabhhhcodes:fix/Retro-batch-9
Open

fix: total accumulation bug, HTML in JS file, orderBtn null crash#953
saurabhhhcodes wants to merge 1 commit into
Anjaliavv51:mainfrom
saurabhhhcodes:fix/Retro-batch-9

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Fixed cart.js: total variable accumulation across calculateBill calls, removed embedded HTML script tags from .js file, wrapped orderBtn in DOMContentLoaded listener

@github-actions github-actions Bot 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 job, @saurabhhhcodes! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. Keep up the excellent work!

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.

1 participant