kwuunnn / pe

0 stars 0 forks source link

Editing group clears all existing groups #11

Open kwuunnn opened 5 months ago

kwuunnn commented 5 months ago

image.png

This contact is in group cs2103t

When entering edit id/E1234562 g/cs2100 to add the same student to another group cs2100,

image.png

The existing groups are cleared. Would be troublesome if I have to retype all the groups. Furthermore it is not intuitive and may lead to bugs when users accidentally clear existing groups unknowingly.

nus-pe-script commented 5 months ago

Team's Response

Our intended behaviour for this command is written clearly in our UG: "Existing values will be replaced by and updated to the new input values.". It is our team's discretion that the edit command can only support that behaviour.

photo_2024-04-21 00.22.11.jpeg

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I think I meant to label it as a FeatureFlaw instead, not a FunctionalityBug.

My apologies.