search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Unable to obtain multiple contact types #611

Open nus-pe-bot opened 3 days ago

nus-pe-bot commented 3 days ago

image.png

Command input: obtain wgtn /number /email

Seems like obtaining multiple contact types is not supported. I think it would be better to indicate it in the user guide at the obtain function section.

image.png

Understand that this question in the FAQ attempts to address it, but from the phrasing of the answer, it seems to only indicate that multiple universities are not allowed.

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

chewycj commented 2 days ago

Team's Response

Thanks for the feedback! We understand your point of view that the FAQ section for obtain could be placed in the UG's obtain function section, hence we would mention it twice in the UG (once in the obtain function section and once in the FAQ) and with better phrasing in the next iteration. It is also worth noting that negative test cases should not be highlighted in the UG and instead it should be highlighted in the manual testing section of the DG. We also believe that this would be a documentation bug instead of a functionality bug as the functionality of obtain still works as how our team expected it to be and it is also unlikely going to affect normal operations of the product.

Duplicate status (if any):

--