HiteshriAcharya / pe

0 stars 0 forks source link

Assigning exercise to workout and renaming workout #2

Open HiteshriAcharya opened 2 months ago

HiteshriAcharya commented 2 months ago

Due to the previous bug i mentioned, when i rename the workout and try to get the info, it has all been deleted. Instead of deleting, you can also merge if they have the same name:

image.png

soc-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Rename workout

If i have added a workout "pull up day" and another "push up day" and i decide to rename "push up day" to "pull up day" which is the same as my first workout name, it should warn me that this name exists and not change it and give me only one output. What if the use has forgotten that this name exists already?

image.png


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

Their Response to the 'Original' Bug

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

No details provided by team.

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: In my opinion, these are not 2 same bugs. Instead, it is one leading to the other. In the first bug, it deletes the workout if I rename to another existing exercise. However, in this one, all the exercises within the workout are deleted, instead of adding it to the existing workout AFTER ASSIGNING.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** I mean you are deleting the entire database of the user just like that when they rename. I don't think this is a "LOW" sorry!