search

liuzehui03 / pe

0 stars 0 forks source link

init function overwrites data #5

Open liuzehui03 opened 2 months ago

liuzehui03 commented 2 months ago

currently in the same timetable if a user init CEG and fills in timetable with courses grades and saves. user can init ceg again and everything added previously would be erased so if this was done accidentally the user, may accidentally delete / overwrite their own data

Whenever the init command is executed, the application should check if there is already existing data for the user's timetable. If data exists, the application should prompt the user to confirm before overwriting the existing data.

image.png

soc-se-bot commented 2 months ago

Team's Response

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`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]