Pedro M. Baeza

Results 2232 comments of Pedro M. Baeza

See my comments in the PR, and it's also conflicting right now in v11.

If you consolidate all modules, maybe it should be called more generically `account_lock`.

@feketemihai can you please rename this for going ahead? We will push the review + merge after that.

I have just merged account_lock_date_update. Is this one still needed?

Can you please add a regression test to prove the problem and its solution?

Create another journal with the same bank account linked. I have yet to check how reconciliation goes with this though.

The same problem then will appear on 10.0. It's better to simply adapt to the new states, that are equivalent in fact.

Not really, it was introduced randomly and we are willing to remove it: https://github.com/OCA/bank-payment/issues/740#issuecomment-856960532