-
-
Notifications
You must be signed in to change notification settings - Fork 600
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
[9.0] [MIG] POS Restricted Customer List #343
[9.0] [MIG] POS Restricted Customer List #343
Conversation
b595701
to
63b56aa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review
Travis is red for lint errors related to other modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't properly test it on runbot, as some other modules in this branch seem to have bad interactions. Here are some comments anyway:
pos_restricted_customer_list/static/src/js/pos_restricted_customer_list.js
Outdated
Show resolved
Hide resolved
pos_restricted_customer_list/static/src/js/pos_restricted_customer_list.js
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe there could be a _get_partner_domain
method but otherwise LGTM 👍
3146ae2
to
b968c45
Compare
9.0 branch is red as Travis could not launch |
@ThomasBinsfeld I fixed 9.0 travis, so you can rebase to get green travis |
@chienandalu Thanks a lot! Rebased. |
* [ADD] New module to only load specific customers in POS. * [FIX] Changes after review - renamed module to pos_restricted_customer_list.
b968c45
to
45b96ff
Compare
This PR has the |
Migration of https://github.com/OCA/pos/tree/8.0/pos_restricted_customer_list.