search

nidhi-nayak / pe

0 stars 0 forks source link

no confirmation message printed for `add savings` #12

Open nidhi-nayak opened 5 months ago

nidhi-nayak commented 5 months ago

image.png

the user is unsure if this command worked or not. printing a confirmation message to assure that the savings were added might be helpful.

soc-se-bot commented 4 months 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

No details provided by team.

The 'Original' Bug

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

Lack of success message when adding savings repeatedly

As shown from the screenshot, when I try to repeatedly add savings for the "salary" category, there is no success message to provide confirmation for the user that the savings has been added. This may lead the user to think that the application is not responding to the command.

image.png

[original: nus-cs2113-AY2324S2/pe-interim#116] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Indeed, this is due to the oversight of me during the implementation. Thanks for reporting this bug.

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 reason]

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