lackmoo / pe

0 stars 0 forks source link

Invalid file naming for CSV accepted #6

Open lackmoo opened 5 years ago

lackmoo commented 5 years ago

This input was accepted as the name of the CSV file to be accepted even with the spacing.

Screenshot 2019-11-15 at 5.14.16 PM.png

nus-pe-bot commented 4 years ago

Team's Response

In regards to this issue, we should have been more clear and specify the naming conventions on the User Guide to not allow spaces. However, the resultant file "random" is still a proper CSV file and still can be opened as per normal, thus we have reduced the severity to Low.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low]. Originally [severity.Medium].

Reason for disagreement: I feel that the severity of this issue should remain at severity.Medium.

Although the resultant file "random" is still a proper CSV file and can still be opened as per normal, it will cause occasioanl inconvenience to users who had intended to save the file name as the name right before ".csv", in this case "hi" in "hi.csv". Instead, as the file name is saved as the word "random" that comes before " hi", users may have a difficult time trying to locate this export file that does not have its intended name, especially if we are talking about a multitude of export files to search from.

Hence, this issue should be of type severity.Medium.


:question: Issue type

Team chose [type.DocumentationBug]. Originally [type.FunctionalityBug].

Reason for disagreement: [replace this with your reason]