kyrenechua / pe

0 stars 0 forks source link

Able to deactivate inactive prescription #12

Open kyrenechua opened 1 year ago

kyrenechua commented 1 year ago

image.png Program does not call an error when prescription is inactive.

soc-se-bot commented 1 year ago

Team's Response

Expected behavior, duplicate

The 'Original' Bug

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

Deactivate command works on inactive items

Deactivate command still works on prescriptions that have already been made inactive.

Screenshot 2022-11-11 at 4.46.46 PM.png


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

Their Response to the 'Original' Bug

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

This is allowed: we want the doctor to be able to ensure that they have de activated something. Performing the command twice has no harm, although one possible improvement could be that we alert the doctor it is already de activated.

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.NotInScope]

Reason for disagreement: [replace this with your explanation]