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] Payments table on the invoice is showing in the print, even if IS POS option is disabled #8802

Merged

Conversation

rohitwaghchaure
Copy link
Collaborator

@rohitwaghchaure rohitwaghchaure commented May 12, 2017

Issue
screen shot 2017-05-12 at 6 23 43 pm

Fixed
screen shot 2017-05-12 at 6 24 32 pm

delete from
`tabSales Invoice Payment`
where
parent in (select name from `tabSales Invoice` where is_pos = 0)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be in safe side, add another condition for payment amount

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert the changes

@rohitwaghchaure rohitwaghchaure force-pushed the sales_invoice_pos_payment_issue branch from 3881a7b to d2ad19e Compare May 16, 2017 05:45
@rohitwaghchaure rohitwaghchaure force-pushed the sales_invoice_pos_payment_issue branch from d2ad19e to e493489 Compare May 16, 2017 05:56
@nabinhait nabinhait merged commit 73456ac into frappe:develop May 16, 2017
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.

2 participants