Skip to content
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

fix: Wrong variable name breaks Cart Items rendering #29400

Conversation

marination
Copy link
Collaborator

Port of #29398
issue:
After adding a variant to cart from the popup , having no image (nothing in template web item too), cart breaks:

jinja2.exceptions.UndefinedError: 'erpnext.selling.doctype.quotation_item.quotation_item.QuotationItem object' has no attribute 'web_item_name'

fix:

  • Correct variable name
  • Test cart items rendering

@marination marination force-pushed the minor-quotation-doc-wrong-var-pre-release branch 2 times, most recently from 2541869 to b27fe43 Compare January 21, 2022 12:39
@marination
Copy link
Collaborator Author

marination commented Jan 21, 2022

I try debugging and everything passes... Cant replicate the failing test either. Plus time crunch for release :(
Screenshot 2022-01-21 at 6 07 33 PM

@marination marination force-pushed the minor-quotation-doc-wrong-var-pre-release branch from dbe3dba to 5ed4b43 Compare January 21, 2022 13:52
@marination marination merged commit f9de807 into frappe:version-13-pre-release Jan 21, 2022
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.

None yet

1 participant