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: Price list rate not fetched for return sales invoice fixed #26559

Merged

Conversation

nemesis189
Copy link
Contributor

When a sales invoice is created with 'Is Return' checked and a price list is selected(other than standard), the items added to the child table wouldn't auto fetch the price list rate. Fixed it by removing the condition that prevented it.

Steps:

  1. create sales invoice with Is Return checked
  2. select a custom price list
  3. add items and check its rate

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 42.947% when pulling 1d2b506 on nemesis189:fix-price-list-rate-not-fetched-in-si into 47f200a on frappe:develop.

@AfshanKhan AfshanKhan merged commit 013b352 into frappe:develop Jul 20, 2021
wikiodisha1 added a commit to wikiodisha1/erpnext that referenced this pull request Jul 22, 2021
When a sales invoice is created with 'Is Return' checked and a price list is selected(other than standard), the items added to the child table wouldn't auto fetch the price list rate. Fixed it by removing the condition that prevented it.

Steps:

    create sales invoice with Is Return checked
    select a custom price list
    add items and check its rate

port for : frappe#26559
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
…pe#26559)

Co-authored-by: Subin Tom <subin-home@Subins-MacBook-Air.local>
Co-authored-by: Afshan <33727827+AfshanKhan@users.noreply.github.com>
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

3 participants