nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Race conditions on adding new patients #1382

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 months ago

Format for bugs (for copy-paste)

Steps to reproduce

Open 2 windows at the same time and add a different patient to each. The model should store both. Close both and open a new window.

Expected

Both of them should be saved.

Actual

Only the first person added was saved.

image.png

image.png


[original: nus-cs2103-AY2324S2/pe-interim#1133] [original labels: type.FunctionalityBug severity.Low]

alfaloo commented 5 months ago

Team's Response

Our product is designed to be used by a single clerk on their own desktop computer at the reception. The scenario you described of having two instances open will never occur and thus will affect none of our target users.

Furthermore, the CS2103 course constraints clearly state that the tP aims to create a single user product (see screenshot below). The issue you raised is not applicable.

Screenshot 2024-04-21 at 8.47.11 PM.png

Duplicate status (if any):

--