Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.0] pos_localstorage #31

Closed
wants to merge 2 commits into from
Closed

[7.0] pos_localstorage #31

wants to merge 2 commits into from

Conversation

OSguard
Copy link

@OSguard OSguard commented Jun 19, 2015

Module that stores the product in browser localstorge for faster loading of pos

// List of product attributes that the POS uses.
var productAttributeList = ['name', 'list_price','price','pos_categ_id', 'taxes_id', 'ean13', 'default_code', 'variants',
'to_weight', 'uom_id', 'uos_id', 'uos_coeff', 'mes_type', 'description_sale', 'description'];

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you set searchGracePeriod and productAttributeList as model vars to allow other modules to overload more easely ?

@legalsylvain legalsylvain added this to the 7.0 milestone Jun 19, 2015
@legalsylvain
Copy link
Contributor

Up ! @OSguard what about our last comments ?

@rafaelbn
Copy link
Member

I guess this PR is ready to be merged. @OSguard could you give me you feedback please?

cc @jcarlosmontoya @JGarcia-Panach your review here could be fantastic please! Thanks!

@rafaelbn
Copy link
Member

rafaelbn commented Jul 5, 2016

Hi @OSguard please if you could review this PR and fix travis we will review it to get it merged! Thanks! 😄

@legalsylvain
Copy link
Contributor

Closing this PR, due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants