search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Commands are not optimized - having to specify the IDs of visitors instead of names #1317

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Steps to reproduce: I tried checking out a visitor using their name but it resulted in an error.

All the commands require us to mandatorily specify a visitor using their ID digits. This is kind of tedious because as the list increases to more than 100 people, it is hard to checkin them our checkout, or find their details, using their id digits solely. It would be easier if names were used to identify them instead of ID digits solely. Both options could be given.

image.png

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

RainyCodeWizard commented 3 years ago

Team's Response

Names are not unique, but ID of the visitors are unique. Hence, checking in and out requires ID of the visitor. So, when you are checking out, you can just ask the ID of the visitor to checkout, so it wont be tedious.

Duplicate status (if any):

--