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

Update facture.php #2525

Closed
wants to merge 2 commits into from
Closed

Update facture.php #2525

wants to merge 2 commits into from

Conversation

frederic34
Copy link
Contributor

If you reopen invoice, it add a payment button to register a payment (negative) by example if your bank reject the first payment

If you reopen invoice, it add a payment button to register a payment (negative) by example if your bank reject the first payment
@frederic34
Copy link
Contributor Author

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Apr 2, 2015
@eldy
Copy link
Member

eldy commented May 21, 2015

If a payment is refused, you can open the invoice, but i think you should instead delete payment (no money move were finaly done) instead of introducing a negative payment.

@eldy eldy added the PR not qualified PR is not qualified (feature not enough requested, duplicate feature or other reason) label May 21, 2015
@frederic34 frederic34 closed this May 25, 2015
@frederic34 frederic34 deleted the patch-14 branch May 25, 2015 11:31
@atm-maxime
Copy link
Member

@eldy, The right way is just as @frederic34 did. When you make a check deposit, the bank puts the money on the bank account and take it back if the check is finally refused. So we should have a positive line followed by a negative one. Just as it's done with withdraw rejection.

I'll work on an easy solution doing this automatically with a button "Check refused" on the deposit slip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed PR not qualified PR is not qualified (feature not enough requested, duplicate feature or other reason)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants