kinjalagarwal1810 / pe

0 stars 0 forks source link

Relate and unrelate command #10

Open kinjalagarwal1810 opened 4 months ago

kinjalagarwal1810 commented 4 months ago

I understand you want to use this command to show who is related to one another. However, the UI does not show this relation anywhere which makes it very difficult to test/use/remember for the user. Perhaps you could add another field that shows the relations someone has so the only way to find the relations is not just with the command showrelated.

image.png

soc-se-bot commented 4 months ago

Team's Response

We have intentionally decided not to implement another field to store the related profiles because it is not scalable. For example, if the employee has 100 related suppliers, there would be the 100 names listed in the field, which can be messy, hard to navigate and hard to pinpoint the profiles that the user wants to locate. Having unrelate and showrelated commands offer a far cleaner way when scaling up relations between contacts.

In addition, we have stated under planned enhancements in the DG that we plan to make the relate and unrelate command success message more friendly (under points 2 and 7). We were prevented from doing so due to the feature freeze, which includes making commands more user friendly. Screenshot 2024-04-22 at 3.00.33 PM.png

Thanks!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Yes, a UI might cut off the relations if there are too many, however did you consider other options like a separate section on your GUI. You can have your main page which is the format of ab3 and add a new right section where if you click on the person you can see all the relations they have on the right side as a list or graphical formats. This would improve the user friendliness of your function greatly.

I am not commenting on the success messages.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** This will cause an issue for most users as anyone who uses the relate function will face this issue.