Open nus-pe-bot opened 1 week ago
Thank you for bringing this issue to our attention. While it is a valid observation that certain products may have typical or logical units of measurement, implementing a feature to validate the nature of the product against its unit of measurement is currently outside the scope of our application. This is due to the complexity and the additional resources required to maintain a comprehensive mapping of product types to valid units, which is not a priority for the current version (v1.6). Additionally, it is reasonable to expect that our users may add products that are unique to their shop, making it impractical to determine or enforce what units of measurement are valid for those specific products.
--
Description:
When entering the
add -d
command, it should check if the item has the corresponding unit of measurement. It does not make sense for an item of typerice
to have the unit of measurementL
because rice is in solid state andL
is reserved for items in the liquid state.Steps to Reproduce:
add -d on/19-12-2022 08:00 s/2 pro/rice q/50 L c/50.20
Expected Result:
The command should fail to execute because rice has no unit of measurement
L
.Actual Result:
The command executes successfully as shown from the screenshot below.
[original: nus-cs2103-AY2425S1/pe-interim#757] [original labels: type.FeatureFlaw severity.Low]