nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Could be confusing for users as the supplier which the good is bought from is deleted, but it still shows on the app.

How to replicate: buy 6 g/apple q/100 delete-s 6

_{[original: nus-cs2103-AY1920S2/pe-interim#2373]}

FangShaoHua94 commented 4 years ago

Team's Response

Intended behavior

The transaction record is a piece of history that should not be affected by current event such as the deletion of supplier.

Eg, think of transaction as receipt. the receipt will continue to exist even when the supplier does not exist anymore.