fy17ohhh / pe

0 stars 0 forks source link

Delete retionship command not working as intended #7

Open fy17ohhh opened 2 months ago

fy17ohhh commented 2 months ago

image.png Throws an error when the corrrect format is inputed

nus-pe-bot commented 2 months ago

Team's Response

The command is working as intended. As said in the error message itself, the command formatting is deleteRelation /UUID1 /UUID2 /relationshipDescriptor. Although you put a slash for the UUIDs, you did not do so for the relationship descriptor 'friends'. However, as we were trying to recreate this error, we realised that this may have been copied directly from the UG example of deleteRelations.

IMAGE 2024-04-21 16:12:49.jpg

We acknowledge that this was a documentation bug. However, seeing as this was only one example that was forgotten and the correct formatting that applied to all the different situations is above:

Screenshot 2024-04-21 at 4.14.14 PM.png

We have put this down as a low-severity bug. It does not affect the normal operations of the application. It is a minor inconvenience that users might face if they copy and paste the example, which in itself might be a rare occurrence as the examples themselves might not work as the UUIDs are not catered to the UUIDs the users would have in their app.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]