Merge lp:~openerp-dev/openobject-addons/7.0-mail_web_analytics-vta into lp:openobject-addons/7.0
Proposed by
Victor Tabuenca (OpenERP)
Status: | Work in progress |
---|---|
Proposed branch: | lp:~openerp-dev/openobject-addons/7.0-mail_web_analytics-vta |
Merge into: | lp:openobject-addons/7.0 |
Diff against target: |
161 lines (+142/-0) 3 files modified
web_analytics_mail/__init__.py (+24/-0) web_analytics_mail/__openerp__.py (+34/-0) web_analytics_mail/static/src/js/web_analytics_mail.js (+84/-0) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/7.0-mail_web_analytics-vta |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Trigaux (OpenERP) (community) | Needs Fixing | ||
Review via email: mp+143111@code.launchpad.net |
Description of the change
Added Google Analytics tracking capabilities to mail module
To post a comment you must log in.
Unmerged revisions
- 8555. By vta <email address hidden>
-
[MERGE] Latest trunk
- 8554. By vta <email address hidden>
-
[FIX] Call super inside overriden methods, -_-U
- 8553. By vta <email address hidden>
-
[ADD] Added new module to track relevant events of mail module in GA
- 8552. By vta <email address hidden>
-
[MERGE] Latest trunk
Could you resubmit this branch for trunk instead ? This
Related to the code :
- should be in autoinstall (?)
- instead of your dictionary in mail_actions, why don't you use the classname as key (eg: 'oe_read': 'Done') ? It will simplify your code
- why using once 'click' and once 'mousedown' ?