search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inconsistent command formats for edit and addMedCon etc #2853

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

edit s1234567a ... does not require i/ prefix before the NRIC but similar commands like addMedCon and delMedCon do; might be confusing for users. i understand the i/ prefix is needed for the new nric if any, but it can be circumvented by using ni/ (new i) for the new index or something similar

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

hweinian commented 2 weeks ago

Team's Response

Thank you for the feedback! It might be confusing for the user due to the inconsistencies in the way the NRIC is used as either a preamble or with a prefix for certain commands, but referencing the user guide resolves this issue. However, we do agree with your concern that the user might be required to remember multiple formats. Thus, we could consider including this in future releases, but it is of low priority as it does not hinder the functionality of the command.

Duplicate status (if any):

--