search

JQChong / pe

0 stars 0 forks source link

Placement of numbering of the sessions is misleading #5

Open JQChong opened 3 years ago

JQChong commented 3 years ago

It took me a while to realize that those are the indices of the sessions. It is documented in the user guide, only under the delete_session command, but not under the overview section.

image.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for pointing this out! Our team understands your concern regarding this issue. However, we do not think that this is considered a bug. The reason that our team included the explanation of the indices only in the delete section was that it was the first instance that required the use of the index in the command.

A room for improvement for us would be to include the explanation earlier on in the user guide to help improve the understandability. But since this is a suggestion, it would not be considered a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I doubt I phrased this as a suggestion. I feel that this is rather important to be addressed under the overview section since this is such a prominent item in the application.