RunjiaChen / pe

0 stars 0 forks source link

Edit still succeeds even when no changes are made #10

Open RunjiaChen opened 5 months ago

RunjiaChen commented 5 months ago

image.png

Description

If I call the edit command without changing anything, success message is still returned

Steps to reproduce

  1. Launch the application for the first time to load initial data.
  2. Run edit command, set the name as the same

    Expected behaviour

    There should be an error message to warn me that no edits were made

    Actual behaviour

    No warnings were made, and the success message is displayed

    Reason for severity

    very minor flaw for users

soc-se-bot commented 5 months ago

Team's Response

Thanks for sharing your feedback on this. This seems to be more of a FeatureFlaw than a Functionality bug as the edit command is working as intended. After considering this, we still believe that adding a warning message specifically for 'no edit made' for the 'edit' command might not be essential compared to warning messages meant for 'delete' and 'clear' commands that affect data removal or reset. The edit function typically involves minor changes that users expect to make without additional alerts. Additionally, the edited information will be shown in the output message as shown in the image you've provided which users can check the changes made.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** I would agree that is a FeatureFlaw, but it is still a bug, as there may be scenarios where the user edits multiple contacts at once, and can be prone to making mistakes. When that happens, an error message should be shown to warn users if they made not make the edits they intended to. Even if you show the updated details to the user, they still need to look scroll through the long list of fields, which may be a hassle