search

TheCSharpAcademy / CONSOLE.PointOfSale

6 stars 15 forks source link

Implement ViewOrderDetails() #51

Closed TheCSharpAcademy closed 1 year ago

TheCSharpAcademy commented 1 year ago

Currently we're showing the details of an order in the ViewOrders method.

. Remove the lift of products from that method. The list of products in an order should only be visible when ViewOrderDetails(), get called. . When shown a list of products, the users need to be asked if they want to go back to main menu or choose an order id to view its details.

brun32700 commented 1 year ago

@TheCSharpAcademy Hello, could you please assign me this issue? I do not have the ability to do it myself. I have already assigned it on the website. Thanks in advance!

TheCSharpAcademy commented 1 year ago

@TheCSharpAcademy Hello, could you please assign me this issue? I do not have the ability to do it myself. I have already assigned it on the website. Thanks in advance!

Done! 😊

brun32700 commented 1 year ago

@TheCSharpAcademy Thanks!

Just want to clarify 2 things.

  1. Inside the ViewOrderDetails() method. Do we need to ask the user for an order id, and then return the order with all the associated products? (This would also involve adding a GetOrder(int id) method inside the KebabController)?

Or

Should all the orders be displayed first to the user using the ViewOrders() method, and then ask the user for an order Id, and return all the associated products?

  1. Currently the Id of created orders is not exposed, shouldn't the id also be shown in the MainMenu.ViewOrders() method?

If it is easier for you, maybe we could arrange a call on Discord, to go over this?

Thank you in advance! :)