search

ramenmen / pe

0 stars 0 forks source link

Blood Type not stored consistently #2

Open ramenmen opened 3 years ago

ramenmen commented 3 years ago

Blood Type retains capitalisation of the command. and is not consistent input: add n/John p/123456789 ic/s1234567a b/o+ Screenshot 2020-11-13 at 4.25.20 PM.png

input: add n/John p/123456789 ic/s1234567a b/O+ Screenshot 2020-11-13 at 4.25.27 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Reason for "rejecting": We want to give the users the flexibility to express the blood type in their preference of using capitalisation or not. Since capitalized or not, blood type is still a valid input that informs the users about the blood type of the relevant patient.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: You could still give users flexibility to express blood type in inputs, but change it to standardised capitalisation when displaying it. I believe that the current way is a cosmetic flaw as most medical records would record blood type with standardised capitalisation, and thus is a bug and should not have been rejected.