AY2425S1-CS2103T-W14-3 / tp

Other
0 stars 5 forks source link

[PE-D][Tester C] Preceding space allowed for certain inputs in name only #213

Closed nus-se-bot closed 6 days ago

nus-se-bot commented 1 week ago

If I were to do add_member n/tag/John Doe r/4-3-301 t/3efqfqef tag/logistics, this is a valid input that will return a person with a name tag/John Doe, but if I do instead add_member n/ tag/John Doe r/4-3-301 t/3efqfqef tag/logistics, it will throw an error as seen in image below. However, if I do add_member n/ John Doe r/4-3-301 t/johndoe123 tag/logistics`, it does add in a person with the name John Doe. Seems to have certain inconsistencies with what is an acceptable name input, could be clarified in the User Guide too. Suggestion is to just not allow the slashes in the input name as I think this will create problems.

As this is my final bug report for this, just want to say overall I think yall did a good job and it is great that most of the commands are designed simple enough for any user to follow along and use. The adding of sessions, tele handles, tags also make the UI very colourful which is nice. Hope my bug reports were helpful to some extent and all the best for your actual practical!

Screenshot 2024-11-08 at 6.03.42 PM.png


Labels: severity.Medium type.FunctionalityBug original: nws321/ped#13

keithxun commented 6 days ago

I do not think this is a functionality bug?