Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unsupported command stated in UG #7

Open
Linxcathyyy opened this issue Nov 12, 2021 · 1 comment
Open

Unsupported command stated in UG #7

Linxcathyyy opened this issue Nov 12, 2021 · 1 comment

Comments

@Linxcathyyy
Copy link
Owner

Linxcathyyy commented Nov 12, 2021

Screenshot 2021-11-12 at 4.57.45 PM.png

Screenshot 2021-11-12 at 4.58.01 PM.png

I tried to use the sample "sortC by/alg o/a" command stated in the UG, however, it turned out that the app did not support this operation.

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 16, 2021

Team's Response

Sort commands are not intended to be able to sort by optional fields as shown in the note section. It's our mistake that we include the example in the UG, however, it's only a minor inconvenience, thus does not affect any other operations. Thus, this should be low severity.

Items for the Tester to Verify

❓ Issue severity

Team chose [severity.Low]
Originally [severity.Medium]

  • I disagree

Reason for disagreement: According to the CS2103T website, the bug actually affects most users and causes major problems for users as this command is not executable at all. However, since the sort command might not be commonly used as compared to add and edit commands, I believe this flaw causes occasional inconvenience to some users but they can continue to use the product. I don't think this happens in rare situations as this command is explicitly stated in your UG. Hence the severity should be medium.

Screenshot 2021-11-17 at 1.35.39 PM.png


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants