← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~sylvain-legal/openerp-pos/7.0_pos_select_customer into lp:openerp-pos

 

Hi again ! 

I did the changes, writing the first part instead of the second part. The result is not the same in the part 1. That breaks the display of the description. Are you sure about what you said ? Please see the screenshot in the links.
Regards.


(TEST 1) -> http://hpics.li/d8c3c94
Possible improvements and fix:
------------------------------
    * This module displays all the customers. That can be long to display
 if there are a lot of customers in database; Display only 80 could be a
 solution;
    * Images of customers are loaded each time; Similar behaviour as
 products image management (with cache) could be developped;

(TEST 2) -> http://hpics.li/b9043e0
Possible improvements and fix:
------------------------------
    * This module displays all the customers. That can be long to display"""
    """ if there are a lot of customers in database; Display only 80"""
    """ could be a solution;
    * Images of customers are loaded each time; Similar behaviour as"""
    """ products image management (with cache) could be developped;
-- 
https://code.launchpad.net/~sylvain-legal/openerp-pos/7.0_pos_select_customer/+merge/228826
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:openerp-pos.


References