nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

For the find feature maybe can group all the find, find-b, find-s (all the find related) together in UG (DUPLICATION) #209

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


Maybe it do not need to show it as separate one cuz it works similar except for the find who only. So maybe can group together, so it would not be too lengthy for the user to read (duplication)


[original: nus-cs2103-AY2223S1/pe-interim#270] [original labels: type.DocumentationBug severity.Low]

wweqg commented 1 year ago

Team's Response

Hi,

Thank you for your suggestion. We try our best to explain everything clearly. The reason we decided to have three sections, one for each sub-type of the find command is that we want to reserve space for future improvement. As you might find out during the PE, buyers, deliverers and suppliers have different attributes and will have more different attributes in subsequent versions, so the prefixes and parameters users can use to find a person will differ depending on which type the person is. For example, users can find a buyer by orders, and that certainly cannot be applied to suppliers. That's why we need different sections for the find command.

Duplicate status (if any):

--