Jolonauh / pe

0 stars 0 forks source link

order command: able to add duplicate orders #11

Open Jolonauh opened 4 months ago

Jolonauh commented 4 months ago

image.png command used: order 7 d/1xRoses c/40 by/23-07-2024 00:00 (entered twice) expected: when adding an order that shares the exact parameters of another order, this could very well be a mistake on the user's end (input error). The application should at least prompt the user to confirm that this is not an error. actual: exact order is added without prompting user.

nus-pe-script commented 4 months ago

Team's Response

Clients may order the same thing on the same delivery date.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I agree that it is a possiblity that the client may want to order the exact same order in two separate orders. However, in a case where an exact input is entered twice, I strongly feel that would more likely than not be an input error by the client instead (e.g. copy and pasted command and entered twice).

For this case, a prompt given by the application would be a more suitable response, as the user would be aware of the extra order. If it was an input error, great! The user is notified and can easily delete the order. If the user intended to make a duplicate order, also great! They are still able to do so. It is a win-win.

I believe this scenario falls under this section provided in the textbook.

image.png