nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incorrect error message when using a wrong parameters that is after email parameters #510

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

steps to reproduce Use a wrong parameter like T/ or f/ edit John Doe p/91234567 e/johndoe@example.com T/ankle pain

output

Screenshot 2024-11-15 at 5.36.01 PM.png

Program throws email invalid format error when it is not the case


[original: nus-cs2103-AY2425S1/pe-interim#720] [original labels: severity.Medium type.FunctionalityBug]

jasmiinee commented 1 week ago

Team's Response

Thank you for raising this issue.

However, in this current iteration, all prefixes are case-sensitive and invalid prefixes will not be recognized. While case insensitivity could improve usability, it does not hinder the primary functionality of the system.

Moreover, it is reasonable to classify this as severity low as this only occurs in a rare situation where optional fields are edited along with the compulsory fields. This feature is thus seen as a lower priority as it requires a lot of effort to implement, hence we decided to use the time to develop other features.

However, we will consider improving the error handling in future iterations.

Duplicate status (if any):

--