nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

`find` and `update` are not cohesive #990

Open nus-pe-bot opened 6 days ago

nus-pe-bot commented 6 days ago

Steps to Reproduce: With the default list, find Irfan followed by update -i 6 -a 1 (update any event with no attendees)

Expected: Irfan to be added to the event

Actual: the first person in the list after running list -p is added instead

Rationale: This prevents users from first finding a person then adding him into the event, instead requiring the user to scroll through their list of persons to find the index of their desired person. Given the target user of being able to manage large extended family networks, this could be inconvenient for users and an improvement would be to use the currently displayed list instead Screenshot 2024-11-15 165758.png


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

michaelyql commented 6 days ago

Team's Response

We agree that there is area for improvements for how the find and update commands should work cohesively.

The intended behaviour should be that update selects users from the filtered list after searching, instead of the list containing all the contacts. However, this will only be an enhancement for the future.

Duplicate status (if any):

--