← Back to team overview

credativ team mailing list archive

Re: lp:~therp-nl/openupgrade-addons/7.0-fix_move_line_currency into lp:openupgrade-addons

 

Review: Needs Information code review & test

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

-- 
https://code.launchpad.net/~therp-nl/openupgrade-addons/7.0-fix_move_line_currency/+merge/186742
Your team OpenUpgrade Committers is subscribed to branch lp:openupgrade-addons.


Follow ups

References