search

kinjalagarwal1810 / pe

0 stars 0 forks source link

Purpose of the relate and unrelate command #17

Open kinjalagarwal1810 opened 4 months ago

kinjalagarwal1810 commented 4 months ago

image.png

The purpose of the above mentioned functions is not clarified. I feel like you could have specified why this function isnecessary for ur target user so as the make it more meaningful this will also help to explain how ur product is suited for the target group you decided.

nus-se-bot commented 4 months ago

Team's Response

Hi! We feel that the purpose of the feature was sufficiently encapsulated in Creates a relation between two profiles in the address book., and intuitive enough to understand the usage and application for a user. It is also a relatively simple and straightforward command that is called a term that means that two contacts will be related together.

Furthermore, the unrelate and showrelated commands help to complement this knowledge with similarly worded commands.

Thank you for your input!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thank you for your response, I understand where you are coming from. However, as a third persons perspective, I believe that short phrase is not enough to encapsulate the purpose, benefit/reason for this feature. I understand it relates 2 Persons, but why would a user want/need to relate 2 Persons. I am not denying it may be a useful feature, however because the purpose of the feature is not stated in the UG, as a potential user of your product I cant imagine a scenario where i would need to use this command.

Creates a relation between two profiles in the address book. This phrase adequately explains the function of the command (Result of what would happen if executed), however i am asking for the benefit/reason/purpose of the command.

Hence, I feel like it is a documentation flaw under this: The explanation is too brief

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** VeryLow is for cosmetic issues. This flaw is `A flaw that causes occasional inconvenience to some users, but they can continue to use the product.` Without understanding the purpose of the feature is is difficult for users to use the product, or maybe they may avoid using the feature which defeats it purpose too.