← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~akretion-team/server-env-tools/web-context-tunnel into lp:server-env-tools

 

Hello Guewen,

sorry about the console.log, removing it.
You made an interesting point about the on_change.
In fact only tests testing modules with the extra fields may fail, not the existing base tests.

Are there so many of such tests? (but yeah lack of test is not a terribly good reason anyway).
Eventually, in OCB, we could modularize the on_change test player so this module have a chance to alter it properly. Then our module will keep being compatible with the commercial branch but at least tests will be properly executed in the OCB branch? Not sure, just thinking...
-- 
https://code.launchpad.net/~akretion-team/server-env-tools/web-context-tunnel/+merge/198599
Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools.


References