← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~camptocamp/account-financial-tools/7.0-add-account_move_batch_validate-lep into lp:account-financial-tools

 

Hi all,

thanks for you reviews, but another question came up just now.

Look at line 181:
i write the job UUID on the move, and then choose only the moves that have no UUIDs. Advantage: we never create many jobs for posting the same move.

Do you think this could be dangerous? Maybe there is a case where we actually need to create a new job to post the same move?

thanks!
-- 
https://code.launchpad.net/~camptocamp/account-financial-tools/7.0-add-account_move_batch_validate-lep/+merge/201187
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.


References