-
-
Notifications
You must be signed in to change notification settings - Fork 584
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
10.0 pos ms issues 3 #1035
10.0 pos ms issues 3 #1035
Conversation
// Multi session without floors | ||
if (!this.config.ms_floor_ids.length) { | ||
return PosModelSuper.prototype.ms_create_order.call(this, options); | ||
} |
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.
seems ok
@@ -60,7 +60,8 @@ def prepare_new_session(self, message): | |||
self.write({ | |||
'run_ID': run_ID, | |||
}) | |||
old_orders = self.env['pos_multi_session_sync.order'].search([('state', '=', 'draft'), | |||
old_orders = self.env['pos_multi_session_sync.order'].search([('multi_session_ID', '=', self.multi_session_ID), | |||
('state', '=', 'draft'), |
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.
👍
@@ -165,7 +166,7 @@ def set_order(self, message): | |||
order = order.create({ | |||
'order': json.dumps(message), | |||
'order_uid': order_uid, | |||
'multi_session_ID': self.id, | |||
'multi_session_ID': self.multi_session_ID, |
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.
👍
@@ -205,7 +213,7 @@ def get_sync_all(self, message): | |||
orders.append(msg) | |||
else: | |||
for order in self.env['pos_multi_session_sync.order'] \ | |||
.search([('multi_session_ID', '=', self.id), ('state', '=', 'draft'), | |||
.search([('multi_session_ID', '=', self.multi_session_ID), ('state', '=', 'draft'), |
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.
👍
1c94832
to
f7f584b
Compare
…hout floors; pos_multi_session_sync - the orders were lost for multiple multisessions working at the same time (itpp-labs#1035) * 🚑 plenty same pos_ID poses caused cross multi-session messaging * 🚑 order loss for multiple multisessions working at the same time * 🚑 error on starting a multisession without floors * 📖 pos ms restaurant readme
No description provided.