search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inability to use \n as values in a log entry #2510

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

Bug Description

Inability to use \n values in log entry as it is regarded as a new line character

Steps to Reproduce

  1. Add a log entry, using addentry, and have it have a \n in the input. For example: \n: Alex

Expected Behavior

The \n should appear.

Screenshots

image.png

image.png

Additional Context

While it is mentioned in the user guide that this is intended behaviour, this might not be optimal for therapists who may use the value \n often in their logs as a way of notation.

The values in the log entry should be escaped.

[original: nus-cs2103-AY2425S1/pe-interim#3421] [original labels: type.FeatureFlaw severity.Low]

slidings commented 19 hours ago

Team's Response

We do agree that \n as plain text will not be able to be inputted by therapist. However, the probability of them needing to use such text with no other alternative seems to be very improbable. We expect therapist sessions to be mainly logged mainly in English. For instances where abbreviation is needed, they are almost always used with a forward slash "/" rather than a computing oriented character backslash "\".

Duplicate status (if any):

--