[BOOM-5292] Order in bo no longer display which employee created it

[prestonBot]

This issue has been migrated from the Forge. Read the original ticket here.

• Reporter: marion_francois
• Created at: Wed, 11 Apr 2018 14:22:19 +0200

Describe the bug

Before in 1.6 version, in the message section we had the employee who created the order. It's not the case anymore in 1.7 version

If there is a problem with the order or payment you do not know who created the order, if you have a telephone pool to take orders you can not distribute the sales commissions according to the employees who created the telephone orders.

Proposed solution

Expected behavior

BO>Order should display which employee has created the Order in the Message block

PS 1.6.1.20

Steps to reproduce

1. BO>SELL>Orders>Orders
2. Create an Order
3. Back to Order List
4. View the latest created order
5. Scroll donw to the Message block
6. See error > No info about the employee who created the order displayed in the message block

Additional infos

• PS version: 1.7.8.5
• PHP version: 7.2

[colinegin]

@TristanLDD @MatShir can you confirm this has not been added to the new order page ?

[TristanLDD]

@colinegin Yes it's confirmed, this was not included in the 1.7.7.

[marsaldev]

@MatShir Basically it should be "re-implemented"? It is still valid this issue?

[hibatallahAouadni]

Ping @PrestaShop/ux-team I proposed a new design :sweat_smile: hope you like it :crossed_fingers: Waiting for your feedback!

Proposed solution

[atomiix]

@TristanLDD friendly reminder :)

[Prestaworks]

@TristanLDD friendly reminder :)

time to "ping" again :)

[TristanLDD]

Hi @hibatallahAouadni, I find your design very coherent with the rest of the interface, I have no problem with using it! I just see two little adjustments to do:

Thank you :)

[Prestaworks]

Should I try to fix this new design in the PR? or will someone with a bit more frontend expertise do that? :)

[hibatallahAouadni]

or will someone with a bit more frontend expertise do that? :)

@NeOMakinG wdyt?

[NeOMakinG]

Should I try to fix this new design in the PR? or will someone with a bit more frontend expertise do that? :)

If you give me the right to push on your fork, I'll do it in the PR directly

[Prestaworks]

Should I try to fix this new design in the PR? or will someone with a bit more frontend expertise do that? :)

If you give me the right to push on your fork, I'll do it in the PR directly

I added you as a collaborator to the fork.

[NeOMakinG]

I pushed, here is how it looks:

[Prestaworks]

Is this going to be merged? (https://github.com/PrestaShop/PrestaShop/pull/28692)

[Hlavtox]

There is a bit more to this.

If you make order in backoffice, it saves both the note and the information about the employee into ps_message.

It also creates a new customer thread, but only adds the order note to it. Not the information about the employee

And if you add a new private message to it in backoffice.

It adds it only to the ps_customer_message.

Result

• Order creation message should be written also to ps_customer_thread.
• It should be decided what to with ps_message table, which is now becoming obsolete, as it is unused.

Ping @PrestaShop/tech-council

Related:

• https://github.com/PrestaShop/PrestaShop/issues/9783#issuecomment-827477060