Joseph31416 / pe

0 stars 0 forks source link

Missing edit functionality for orders #11

Open Joseph31416 opened 7 months ago

Joseph31416 commented 7 months ago

Command List addmem : Add a member additem : Add item addmempts : Add membership points addorder : Add order addpts : Add redemption points clear : Clear all members delmem : Delete a member delitem : Delete item edit : Edit member details exit : Exit the program find : Find members help : Access user guide list : List all members redeempts : Redeem points seeddata : Seed data

There isn't an edit command for order. Since there is no delete command either, there is no way to modify an existing order. This can be rather severe as the user could have made a typo or there might be changes to the order by the customer after the order has been entered.

nus-se-bot commented 6 months ago

Team's Response

Thanks, you are right in saying there is no way to modify an existing order now! But in our planned enhancements, we plan to add the delete order functionality which will solve this issue! We also understand that an "edit order" command could still be useful even with a delete order command, but the time saved with that command is minimal and hence we decided to not include that for now.

  1. Add a delete order command: Currently, if a user accidentally adds a wrong order, the only solution is to delete the member using delmem and add the member back using addmem. This is a lot of trouble cause by a simple mistake. As such, we should have a command that allows deletion of orders, which likely works off indexing the orders of a person. For example delorder n/John 2 deletes the second order of John 2. Of course to complement this feature, the index of orders should be displayed in the GUI. Points awarded in the deleted order should also be revoked.

    Items for the Tester to Verify

    :question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]