Code review comment for lp:~akretion-team/server-env-tools/web-context-tunnel
Revision history for this message
| Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
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.