search

eugenetyc / pe

0 stars 0 forks source link

Error message for editp sales #10

Open eugenetyc opened 4 years ago

eugenetyc commented 4 years ago

editp sales should not state money, but rather sales.

image.png

nus-pe-bot commented 4 years ago

Team's Response

The fundamental basis of sales is that it is a type of Money, hence money is stated. Both usage of 'sales' or 'money' are understandable by users so I don't see why this is an issue. This is not a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I think the team is being overly technical. I came into the PE and tested their app thinking Sales meant the number of units sold. In the real world (not the overly technical one), there are many ways of presenting Sales figures. It could be in terms of number of units, number of dollars, number of batches in a specific time period, sales rates, and etc. Therefore, it is not necessarily always money, like what the team used in their refuting.

This is therefore a feature flaw, because they are assuming that the users think the same as them (they assume the users think sales implies the overall monetary value of sales rather than other representations like number of units sold), which is not always true in the case of their users. Moreover, this is marked low because, while the app can still functions, the users need to convert number of units sold manually to total monetary value manually outside of the app before inputting it into the app, incurring some inconvenience. Therefore, this may not address the target audience exactly well as required by CS2103/T.

image.png