← 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

 

Review: Needs Fixing

It could deserve a test.

You should remove the console.log()

What about the yaml tests? They use the views to apply automatically the onchanges, so the tests will fail because they won't take the extra attributes in account.
-- 
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