search

HubRise / website

0 stars 4 forks source link

[ORDERLINE] Opening Hours update #939

Open janaina-wittner opened 6 months ago

janaina-wittner commented 6 months ago

@antmonnier, this is ready for pulication. If you can do it now, I will translate it.

IMPORTANT NOTES:

UI text "Opening hours can be sent to your ordering and delivery platforms. Check platform support." should be replaced by "Opening hours can be sent to online ordering solutions. Check online ordering support." We cannot limit this on online ordering and delivery platforms. Eventually, LivePepper and the likes will also integrate this.

UI error message "Time periods overlap, consider merging them" could be replaced by "Time slot overlap" It's in read, everyone will understand that it should not be so, especially if the Save button is greyed out. We refer to time slots and not periods in the documentation. I think time slot is better. Let's keep the UI and the online documentation terminology consistent.

The documentation says that Time Slots start and end times must be configured in chronological order. Be aware that no error message displays when this rule is not respected.

Screenshot 2023-12-21 at 10 26 12
antmonnier commented 4 months ago
  1. Not sure about this one, I don't want OrderLine to be too visibly focused on food outlets. The current message "Opening hours can be sent to your ordering and delivery platforms" is more generic yet I think users will not be confused.
  2. I've made the changes in OrderLine (not yet deployed)
  3. It sorts time slots on save.