search

YANYAN-w / pe

0 stars 0 forks source link

No change to person should return some warning message #2

Open YANYAN-w opened 1 week ago

YANYAN-w commented 1 week ago

In the edit command, if the person's particular is not edited, it would be better to show some warning to the user that no change is done to the existing person. The following screenshots are generated by executing view then edit 1 n/Alex Yeoh image.png

image.png

nus-se-bot commented 5 days ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Rejected. The feedback only shows the details of the edited buyer. It is not expected to check if there is any actual change after the edit is executed. Moreover, it is not essential as long as it reflects the correct result of the edit command execution.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]