search

marcuslowhuiyu / pe

0 stars 0 forks source link

Unable to alter current stock via CLI #13

Open marcuslowhuiyu opened 2 years ago

marcuslowhuiyu commented 2 years ago

image.png

image.png

image.png

While it is mentioned that the app is optimised for Command Line Interface, there is currently no way to increment or decrement the stock of an item via the CLI, and this feature can only be done in the GUI. This goes against the constraint of the tP of having typing preferred interactions.

nus-se-script commented 1 year ago

Team's Response

Reason for Response

The 'Original' Bug

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

Violates CLI project constraint

image.png

There is no option to do this via CLI command.

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

Their Response to the 'Original' Bug

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

Reason for Response

  • editStock command aims to supplement what we have in the GUI, where the user can still modify currentStock levels.
  • Our group went with implementing the increment and decrement feature because we felt that it would be a good to have feature.

    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: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is clearly stated in the project constraints that it should be typing preferred. Even if the functionality is offered through the GUI, the bare minimum requirement is that said functionality is available in through the CLI as well.

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: Goes against tP constraints.