germainelee02 / pe

0 stars 0 forks source link

UG says list for language/tag is case sensitive, but it is not #10

Open germainelee02 opened 1 year ago

germainelee02 commented 1 year ago

In the UG, it says that the list for tags are case sensitive. But when I tried, it clearly is not case sensitive. This is the result when I type list t/FRIENDS.

Screenshot 2023-04-14 at 2.48.49 PM.png

Bernice Yu still comes up, even though her tag is lower case

nus-pe-script commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Wrong information. given in the UG

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


Pretty sure it should be case insensitive.

Screenshot 2023-04-14 at 3.16.08 PM.png


[original: nus-cs2103-AY2223S2/pe-interim#4140] [original labels: severity.Medium type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Even in the case that the user takes the information to be accurate, the usage of this command is unlikely to be affected as if the user treats the fields as case-sensitive, they will still be able to use this command effectively. Hence, this DocumentationBug is accepted at a Low severity.

To elaborate, Language is case-sensitive, while Tag is case-insensitive.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]