isotope / core

Core repository of Isotope eCommerce, an eCommerce extension for Contao Open Source CMS
https://isotopeecommerce.org
135 stars 107 forks source link

Bestellungen Sendsnummern nicht nutzerfreundlich #2219

Open bennyborn opened 3 years ago

bennyborn commented 3 years ago

Ich finde es zwar toll das der Core nun ein Feld für Sendungsnummern mitliefert aber von der Usability her ist das nicht schön gelöst.

Von unseren eigenen Kunden wissen wir das diese eigentlich nur in eine Bestellung rein gehen, die Sendungsnummer eingeben und den Status auf "versendet" setzen. Über die Bestellstati wird dann eine entsprechende Notification versendet.

Jetzt muss man den Status ändern, die Checkbox Benachrichtigung senden anklicken, dann die korrekte Benachrichtigung auswählen und natürlich die Sendungsnummer(n) hinterlegen.

Die zur Auswahl stehenden Benachrichtigungen sind hierbei nicht limitiert. Ich könnte aus Versehen auch die Benachrichtigung für "Neue Bestellung" auswählen.

Gerne hätten wir einfach in unserem Fall das Feld für die Sendungsnummern aus der Subpalette raus gezogen, allerdings wird der Simpletoken ##order_status_tracking_numbers## wohl nur befüllt wenn auch der Haken bei Benachrichtigung senden gesetzt wird.

Ich sehe da zwei Optionen:

  1. Das Feld Sendungsnummern rausziehen und unabhängig von Benachrichtigung senden immer ans NC leiten
  2. Das Feld Benachrichtigung limitieren auf einen bestimmten Benachrichtigungstyp der eben nur in diesem Dropdown zur Verfügung steht
haendersonfox commented 3 years ago

hallo benny, wo kann ich den dialog denn sehen? sendungsnummern?

lg. alex

bennyborn commented 3 years ago

In der aktuellen Isotope-Version (2.7.1) wenn man eine Bestellung im Backend bearbeitet.

image

fritzmg commented 3 years ago

Jetzt muss man den Status ändern, die Checkbox Benachrichtigung senden anklicken, dann die korrekte Benachrichtigung auswählen

You don't have to use that. The notification of the order status will still be sent. If not then it may be a bug?

bennyborn commented 3 years ago

You don't have to use that. The notification of the order status will still be sent. If not then it may be a bug?

Sure, I don't have to, the notification would be sent nonetheless but then again I prefer my orders having a proper status ;)

fritzmg commented 3 years ago

I don't know what you mean, the process is still the same in 2.7 as it is in 2.6. You edit the order, change the order status and save. This will send the new status' notification, you do not have to use the optional setting Send a notification.

bennyborn commented 3 years ago

Okay, misunterstood your point here. My problem was that you can not set a tracking number for an order without activating the checkbox Benachrichtigung senden (and of course selecting a specific notification then). The value in the field Sendungsnummern will only be forwarded to the NC if this option is checked.

That's the problem:

https://github.com/isotope/core/blob/f22f6e3a27db8d870e8daeb6923b8930a9acee03/system/modules/isotope/library/Isotope/Backend/ProductCollection/Callback.php#L697-L698

The token order_status_tracking_numbers is only populated if sendNotification is set.

https://github.com/isotope/core/blob/f22f6e3a27db8d870e8daeb6923b8930a9acee03/system/modules/isotope/library/Isotope/Backend/ProductCollection/Callback.php#L708

fritzmg commented 3 years ago

Ah I see. Yes, there does not seem to be a reason for the tracking number to be tied to the Send a notification option.

aschempp commented 3 years ago

The reason is that its meant to be used to explicitly to send the tracking numbers to the client. Select the "shipped with trakking number" notificatation and enter the respective details. I think using the order status to send a notification is just a leftover of "how it was done previously" 🙃

bennyborn commented 3 years ago

Select the "shipped with trakking number" notificatation

Then you should at least limit the type of notification that can be chosen. This is currently absolutely designed to cause problems.🤷

contaoacademy commented 2 years ago

I vote for repositioning the field directly to the first level of the order. I find the mail dispatch in connection with the status change very convenient, and currently always have to add my own "tracking field".