← Back to team overview

openerp-dev-web team mailing list archive

Re: [Merge] lp:~openerp-dev-web/openobject-client-web/group_by into lp:~openerp-dev/openobject-client-web/trunk-dev-web

 

> i used [id={some_value}] that is for matched id$={some_value}.

In some places it's an id$= selector which is normal, but there are also id= calls like line 317 `var total_groups = jQuery('table[id="'+this.name+'"]').attr('groups')`.

Also, PyCharm tells me `group_id` variable is not used, can you check? And a few lines are missing ending semicolon (like 317 and 320)
-- 
https://code.launchpad.net/~openerp-dev-web/openobject-client-web/group_by/+merge/25140
Your team OpenERP SA's Web Client R&D is subscribed to branch lp:~openerp-dev/openobject-client-web/trunk-dev-web.



Follow ups

References