EugeneChanJiajun / pe

0 stars 0 forks source link

Initialising a new major deletes the entire saved list from before. #5

Open EugeneChanJiajun opened 7 months ago

EugeneChanJiajun commented 7 months ago

When I try to initialise a new and back to the original one, the entire saved plan from before is gone. I am not sure if this is intended but there is not warning that the list will be deleted.

image.png

nus-pe-bot commented 7 months ago

Team's Response

This is intended for it to overwrite the current timetable. We have a function for users to create multiple timetables if they want, and each of these timetables will not affect each other.

However, we do agree that it would be better to have a warning message. This will be a duplicate of the "original" issue #1034.

The 'Original' Bug

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

Initiating should come with a data-wipe warning

image.png

Users that might unintentionally reinitiate a major instead of changing timetable can have their edits deleted.

image.png

This shows the removed course reappearing again after the unintentional init,

Improvement: Re-initialising should come with a warning followed by a Y/N confirmation that the user acknowledges the data will be wiped.


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

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: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** After consideration I feel the severity type should still be a severity.medium as it is not stated anywhere that there will be a data wipe after a new initialisation. The main point of a planner is so that users can make as much changes as they want without fear of changes or additions being deleted. If there is no output warning in the program or written warning given in the UG, it would have failed user requirements on what a planner should be and can accomplish.