dwsc37 / pe

0 stars 0 forks source link

Unable to create backups with / in the description #2

Open dwsc37 opened 1 week ago

dwsc37 commented 1 week ago

Screenshot 2024-11-15 at 4.29.26 PM.png

Screenshot 2024-11-15 at 4.30.18 PM.png

Some examples above^

Expected Either

Actual A generic failure message is given.

This could potentially cause problems, as users who want to use / in their description are not given any indication as to why there is an error

nus-pe-script commented 4 days ago

Team's Response

While more descriptive error messages could improve user experience, the generic error message suffices to inform users of an issue. Enhancing the error message is a possible improvement, but it is not critical for the app's core functionality, as users can easily modify their input to comply with naming conventions.

ClinicBuddy focuses on managing patient records and backups efficiently. Spending resources to refine error messages for edge cases, like restricted symbols in backup names, is a lower priority compared to other core functionalities.

Allowing the creation of backup files with the / symbol in their names is impractical as the / character is reserved as a directory separator in most operating systems (e.g., Linux, macOS, and Windows). Supporting such behavior would require additional parsing and sanitization logic to replace or escape the / symbol, which complicates the implementation unnecessarily.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]