Code review comment for lp:~therp-nl/openupgrade-addons/7.0-fix_move_line_currency

Revision history for this message
Sylvain LE GAL (GRAP) (sylvain-legal) wrote :

It's OK for me.

However, when testing (rapid test), I didn't have the case with account_move_line with currency defined in 6.1. (The old values was Null).
What is the user case to have account_move_line with currency_id set ?

@Reviewers : the new constraints is _check_currency_company in account/account_move_line.py

review: Needs Information (code review & test)

« Back to merge proposal