search

kirangeofran / pe

0 stars 0 forks source link

Disallowing symbols in Medicine name input #4

Open kirangeofran opened 4 months ago

kirangeofran commented 4 months ago

The medicine name input does not allow for special characters such as '.'. This could limit the user in the types of medication they could input into the system image.png

nus-pe-bot commented 4 months ago

Team's Response

Thank you for reporting your encounter you had with MediTracker.

A medication with name containing numbers and special characters is not appropriate. Hence, our prompt specifying Please enter a proper medication name is to address this issue.

We also encourage you to take a look at all our examples which we have provided in our UG. None of them, including our sample data, contain medication names which consists of such characters.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Unable to take in numbers for medication name

The medication name input is unable to take in numbers. This is not specified in the UG and is a feature flaw as it should be able to take in any inputted medicine name, so as to not, limit the medications that the user can input.

image.png

image.png

[original: nus-cs2113-AY2324S2/pe-interim#1408] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for reporting your encounter you had with MediTracker.

it should be able to take in any inputted medicine name, so as to not, limit the medications that the user can input.

The statement which you have made above is incorrect. We do not limit the medications which user can input. However, we do not accept medications with inappropriate names. A medication with name containing numbers and special characters is not appropriate. Hence, our prompt specifying Please enter a proper medication name is to address this issue.

We also encourage you to take a look at all our examples which we have provided in our UG. None of them, including our sample data, contain medication names which consists of such characters.

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 response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]