Pedro M. Baeza comments

Results 1905 comments of


                                            Pedro M. Baeza

trafficstars

[12.0][ADD] account_payment_intransit

Well, don't you think this can be something generic? Is the problem for moving this to get it merged here?

[12.0][ADD] account_payment_intransit

But no reviews for now. Your team can review the PR (or yourself), and having 2 reviews, no problem in merge it here.

[12.0][ADD] account_payment_intransit

Of course! Seeing you very soon :)

[13.0][BUG] account_financial_report: Incorrect JS widget dependency

Are you able to provide a fix for this? cc @JordiBForgeFlow

[12.0] account_payment_partner: bank is never set on customer invoice

That's the idea, to let the payment order algorithm to work.

[12.0] account_payment_partner: bank is never set on customer invoice

Yes, there is. Call it debit order if you want.

[12.0] account_payment_partner: bank is never set on customer invoice

`account_payment_partner` already includes a modification in the invoice report for showing the bank account with the preferences set in the payment mode. For Switzerland QR, you need to add specific...

[12.0] account_payment_partner: bank is never set on customer invoice

I don't think it should be conditioned by SEPA DD, but the previous behavior was simply not correct, assigning a bank account that is not going to be used at...

[12.0] account_payment_partner: bank is never set on customer invoice

Is there an option to compatibilize both problems in the base module?

Better use move_id.ref or move_id.payment_reference in payment order ?

I would say it depends in: - It's an invoice or an entry. - It's for customer or for supplier. If you want to propose a pull request handling this,...