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

10.0 pos ms issues 3 #1035

Merged
merged 4 commits into from
Jun 27, 2019
Merged

Conversation

KolushovAlexandr
Copy link
Contributor

No description provided.

// Multi session without floors
if (!this.config.ms_floor_ids.length) {
return PosModelSuper.prototype.ms_create_order.call(this, options);
}
Copy link
Collaborator

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'),
Copy link
Collaborator

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,
Copy link
Collaborator

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'),
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@ilmir-k ilmir-k merged commit e024746 into itpp-labs:10.0 Jun 27, 2019
Ommo73 pushed a commit to Ommo73/pos-addons that referenced this pull request Aug 5, 2019
…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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants