JianJiaT / pe

0 stars 0 forks source link

No command to display total revenue of inventory #3

Open JianJiaT opened 7 months ago

JianJiaT commented 7 months ago

Absence of command to display total revenue of inventory (despite there being a command to display total profits) makes it inconvenient for the user to find said revenue . The user would have to exit and restart the application to see the total revenue

image.png

nus-se-script commented 7 months ago

Team's Response

Duplicate of #1646, as the resolution of issue #1646 will also entail the resolution of this current issue.

Description for #1646

Indeed, this is a missing feature that we were actually planning to implement in the future, and we accept that this is a slight flaw of our profit-reporting feature.

It was left out of scope for the current iteration of the product, as we figured it was more crucial for us to simply display the overall profits earned (which is essentially revenue - cost). We believed that a typical user would be more concerned with their overall profits, instead of just their costs or revenues alone.

Additionally, as you've mentioned, users can already view the cost/revenue of their inventory. However, they'll need to exit the application and restart it in order to view these statistics.

While we agree that this can pose an inconvenience for users, we reckon that the lack of this feature will only cause rare, minor inconveniences for users. We believe that such features will be less used (as compared to our main features/functionalities for inventory management), and thus, the lack of this feature should not severely impact a user's ability to use our application.

Therefore, we would like to seek your understanding to reduce the severity of this bug to Low instead. Thanks for reporting this bug.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

No command to display total cost of inventory

Absence of command to display total cost of inventory (despite there being a command to display total profits) makes it inconvenient for the user to find said cost. The user would have to exit and restart the application to see the total cost

image.png


[original: nus-cs2113-AY2324S2/pe-interim#1451] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Indeed, this is a missing feature that we were actually planning to implement in the future, and we accept that this is a slight flaw of our profit-reporting feature.

It was left out of scope for the current iteration of the product, as we figured it was more crucial for us to simply display the overall profits earned (which is essentially revenue - cost). We believed that a typical user would be more concerned with their overall profits, instead of just their costs or revenues alone.

Additionally, as you've mentioned, users can already view the cost/revenue of their inventory. However, they'll need to exit the application and restart it in order to view these statistics.

While we agree that this can pose an inconvenience for users, we reckon that the lack of this feature will only cause rare, minor inconveniences for users. We believe that such features will be less used (as compared to our main features/functionalities for inventory management), and thus, the lack of this feature should not severely impact a user's ability to use our application.

Therefore, we would like to seek your understanding to reduce the severity of this bug to Low instead. Thanks for reporting this bug.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: This is not a duplicate as the issues are not caused by the same defect and it is possible to solve the two issues independently, for instance by adding a cost command and a revenue command