nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Currency for Price is fixed #160

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

image.png

PRICE does not require you to include the currency. Only include the value.

But I might have items with different currencies in my store. This app will not allow me to specify 2 different currencies and I cannot keep track of my inventories well. It will be a chaos for me to convert all the prices to the same currency because in my stores, I have 2 different currencies


[original: nus-cs2103-AY2223S1/pe-interim#208] [original labels: type.FeatureFlaw severity.Medium]

RichDom2185 commented 1 year ago

Team's Response

While a valid point that FoodRem technically does not support multiple currencies, in the rare use case that this needs to be done (hence should not be medium severity), users can use the remarks field to specify which currency the item is stored in, as it is an open text field.

While this will make any price comparisons less meaningful (as prices are compared by the numerical price value, assuming same currency), we argue this is not in scope, as:

Moreover, the main reason users are using this application is to measure the value of the items in their inventory. We have two choices, first is to create a way for a user to key in what is the current prevailing exchange rate (which fluctuates daily) However, since this is an inventory management application, we are not focusing on exchange rate. It is intended for price to be fixed.

Duplicate status (if any):

--