nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Slightly confusing documentation #1387

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

When a stock is deleted, the unique id is deleted as well. There is no mention of ways to recover this stock ID. When stock is deleted, there should be a way to get more stock and recover this unique id stock. It was found that the id of the stock that got deleted can be found in the data file. This should be said in the UG to the user as well.

_{[original: nus-cs2113-AY2122S1/pe-interim#1432] [original labels: severity.Low type.FeatureFlaw]}

a-tph commented 3 years ago

Team's Response

We did not state this in the UG as there is no need for users to manually recover this stock id. If they intend to delete it, the record should be deleted on the user's perspective.

It will only be restored when deleteprescription command is ran with a medicine stock that has been deleted or has its name updated.

As this implementation is quite technical, we stated in our reasons in our DG but not in our UG to prevent confusing users with our backend implementation.

Duplicate status (if any):

--