search

BransonNg / pe

0 stars 0 forks source link

Does not check whether csv is present before overwriting #9

Open BransonNg opened 4 years ago

BransonNg commented 4 years ago

exportAttendance function does not check if filename already exists and immediately replaces existing file

nus-pe-bot commented 4 years ago

Team's Response

As mentioned in the UG, the csv file will be written to the filepath provided by the user, hence we have faith in the user providing the correct details. Additionally, as a TA, the user will not need multiple attendance files as there is only a Single Source of Truth with regards to whether the student attended the tutorial or not. Hence we do not believe that the user needs an additional burden of providing a different filepath when exporting attendance.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]