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

refactor: In Quotation Item, discount_and_margin section should have same collapsible_depends_on as other similar DocType (Sales Order Item,Sales Invoice Item,...) #37272

Merged

Conversation

FHenry
Copy link
Contributor

@FHenry FHenry commented Sep 27, 2023

Please provide enough information so that others can review your pull request:

Actually on Quotation Item when you fill a discount % or choose a margin type, the section collapse on blur event

Explain the details for making this change. What existing problem does the pull request solve?

Set the same collapsible_depends_on

eval: doc.margin_type || doc.discount_amount

on Quotation Item, field: discount_and_margin

as it is already the case on

"Sales Invoice Item": field discount_and_margin
"Purchase Order Item": field discount_and_margin_section
"Purchase Receipt Item": field discount_and_margin_section
"Delivery Note Item":field discount_and_margin
"Sales Order Item":field discount_and_margin

no-docs

backport of #37252

…same collapsible_depends_on as other similar DocType (Sales Order Item,Sales Invoice Item,...)
@rohitwaghchaure rohitwaghchaure merged commit 21c1c71 into frappe:version-14-hotfix Sep 29, 2023
9 checks passed
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 14.42.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@FHenry FHenry deleted the version-14-collapse_discount branch October 4, 2023 05:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants