ryanmitchell / ti-ext-kitchendisplay

Other
14 stars 8 forks source link

Kitchen Display customization settings #12

Closed bingo-bonzai closed 3 years ago

bingo-bonzai commented 3 years ago

It will be Good to have Kitchen Display settings for 1) Selecting the visible status options (i.e. Prep, Ready, Complete, etc) 2) Selecting Status value to remove the order from the Kitchen Display (like Completed in current state) 3) Similar to Auto Print, an Auto refresh option may be provided using Broadcast events. This may be triggered based on any order status change.

Also instead showing Ready orders in the Kitchen Display (which is not the right place), they may be moved to a Delivery Queue list towards the extreme right side of the Kitchen display. This list can be a simple list displaying customer name, order number and order value. Attaching an edited screenshot for reference.

image

ryanmitchell commented 3 years ago

Yep I fixed it after i messaged

roydeanjr commented 3 years ago

I am good with this! Great job! image

roydeanjr commented 3 years ago

image

roydeanjr commented 3 years ago

@bingo-bonzai @wakey18 @BreakSecurity I am satisfied with this extension and it solving all of the previous issues plus additional issues raised subsequent to testing. Do you all have any additional concerns or wishes that need to be addressed?

roydeanjr commented 3 years ago

I created this issue in the TI repository: https://github.com/tastyigniter/TastyIgniter/issues/560

roydeanjr commented 3 years ago

Tested button and works now. Thank you tons!!!

BreakSecurity commented 3 years ago

Didn't have the time to test yet. But i noticed that order status changed from kitchen display the notification to the customer is NOT sent.

roydeanjr commented 3 years ago

@BreakSecurity This is happening even when the default in the statuses page is set to notify customer?

roydeanjr commented 3 years ago

@ryanmitchell Is it possible to use the status color that is already set up in the status table?

ryanmitchell commented 3 years ago

I decided to let it be changeable by the user

BreakSecurity commented 3 years ago

@BreakSecurity This is happening even when the default in the statuses page is set to notify customer?

Yep.

immagine immagine

roydeanjr commented 3 years ago

Hmmmm.....is this on all status changes or just one?

My system is set up to use sendmail in the settings and I am getting email for all status changes, included a status I just set to notify the customer. Did you check your junk/spam folder? image

roydeanjr commented 3 years ago

There was a time when the email did not get sent. Ryan fixed that and this new v2 works great for me. Looking forward to hearing how it works for the rest of you. Ryan did a great and expediant job!

roydeanjr commented 3 years ago

@ryanmitchell Any idea what it would take to make the address hot so clicking it opens the mapping / driving application? In your spare time!

BreakSecurity commented 3 years ago

There was a time when the email did not get sent. Ryan fixed that and this new v2 works great for me. Looking forward to hearing how it works for the rest of you. Ryan did a great and expediant job!

I still have version 1 that's way I will upload this v2 and try

ryanmitchell commented 3 years ago

@roydeanjr {{ 'https://www.google.com/maps/dir/?api=1&destination='.urlencode($order->address) }} should do it

ryanmitchell commented 3 years ago

@roydeanjr when you get some time would you mind re-working the readme for version2? We'll need to update the marketplace listing too as theres a lot more in this than the original

roydeanjr commented 3 years ago

Yes, I will do that. Not sure when, but I'll get it done.

BreakSecurity commented 3 years ago

Can I share some ideas here on v2?

ryanmitchell commented 3 years ago

I would prefer code :) But go ahead!

On 12 Nov 2020, at 18:27, BreakSecurity notifications@github.com wrote:

 Can I share some ideas here on v2?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

BreakSecurity commented 3 years ago

So I was wondering on how to manage the following scenario: 1 restaurant, 2 kitchens (1 for fish & 1 for meat - random example), n delivery man...

I create 3 + n delivery man views: 1) General view (Cancel or accept orders) 2) Fish kitchen view (view only fish in each order accepted) 3) Meat kitchen view (view only meat in each order accepted) 4) n Delivery view (display address and name of customer has one button: delivered/completed)

To handle the following scenarios there are some problems: 1) Each kitchen will receive only a part of the order, and will complete its task at different times; once they have completed their task they should remove it from their display without removing the order from the displays of the others kitchens (it is still the same order). 2) The order should appear on delivery view only if completed by both kitchen.

To do this each order should have at the same time more status (or flags) like prepared_by_fish_kitchen & prepared_by_meat_kitchen. So delivery view display only orders completed by both kitchens. This actually is not possible with status, correct me if wrong...

Do you have any idea how to do about it?

ryanmitchell commented 3 years ago

There is no way of handling multiple concurrent statuses in the system, and its beyond the scope of what this extension is intended for really.

A work around that might work is to not let the fish/meat etc views mark as completed (take away the button) and have a view for the person who brings all the orders together for service where they get marked as completed and moved to the delivery view.

roydeanjr commented 3 years ago

There is no way of handling multiple concurrent statuses in the system, and its beyond the scope of what this extension is intended for really.

A work around that might work is to not let the fish/meat etc views mark as completed (take away the button) and have a view for the person who brings all the orders together for service where they get marked as completed and moved to the delivery view.

What does the code look like to make the Orders with initial status a multi-select option like the Show menu items from categories can select multiple categories. The assumption could be made that if multiple are selected that the test is AND rather than OR, correct?

ryanmitchell commented 3 years ago

@roydeanjr while that field can become a multiple select, any order within TI only can have 1 status at any time, so ANDing or ORing dont apply in this case.

roydeanjr commented 3 years ago

I can see that scenario working, however, only with an intermediary. A supervisor who is plating could have a button to put the order back on the display of the station that is not finished by modifying the status to a status that is not common between the two stations, i.e. fish_only_station or meat_only_station. Present two views on those stations; one for whole order with Accepted status, the other for the _only_station status. Your "fish_ready" and "meat_ready" statuses should put the order on the "Expediter/Plating" view for completing that task so if the order shows up there they have the button option to move it back to the station that is not finished.

A workaround, but could possibly work? And, none of this status movement would notify the customer or the delivery driver.

roydeanjr commented 3 years ago

@roydeanjr while that field can become a multiple select, any order within TI only can have 1 status at any time, so ANDing or ORing dont apply in this case.

Thank you for the quick reply!

I did not look at the code, but assumed that if the order status is in the list of selected status to display that it would be ANDing or ORing. Maybe it is just a "in set" test? If so, then it could work?

Which protien usually finishes first? Maybe make the meat station show if the order is fish_complete OR received?

roydeanjr commented 3 years ago

@roydeanjr when you get some time would you mind re-working the readme for version2? We'll need to update the marketplace listing too as theres a lot more in this than the original

Readme updated as well as map link added next to the address. Not sure how to not do the map link if the order is a pickup order. Figured that any delivery order would have a legit address and pickup orders would not show on a delivery view.

ryanmitchell commented 3 years ago

Thank you.

Lets revert the name back to Kitchen Display for now - I appreciate it does more than that now but it keeps it easier with updates.

I’ll sort out the pickup order address link.

On 23 Nov 2020, at 16:37, roydeanjr notifications@github.com wrote:

@roydeanjr https://github.com/roydeanjr when you get some time would you mind re-working the readme for version2? We'll need to update the marketplace listing too as theres a lot more in this than the original

Readme updated as well as map link added next to the address. Not sure how to not do the map link if the order is a pickup order. Figured that any delivery order would have a legit address and pickup orders would not show on a delivery view.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ryanmitchell/ti-ext-kitchendisplay/issues/12#issuecomment-732277819, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMVO6OURC4W7CSE24XTITSRKFVVANCNFSM4SYTFKNQ.

ryanmitchell commented 3 years ago

Closing this off as the version 2 branch is now complete