search

Bisceto / pe

0 stars 0 forks source link

Tagging videos not case insensitive #5

Open Bisceto opened 1 year ago

Bisceto commented 1 year ago

Only tested for tagging videos, but it can apply for the other commands when a name is needed.

I have a video "Vid 1" in "Week 1" lecture of CS2040SS. When i key in the command "tag vid 1 /tags intro", it says that the video does not exist.

image.png

While it follows the functionality, it is not case sensitive which can be an inconvenience for users as they may forget to use capitals etc.

soc-se-bot 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]

Module code and lecture name are case sensitive

Module code and lecture name are case sensitive. This makes it more difficult to input them.

[original: nus-cs2103-AY2223S2/pe-interim#3930] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Thank you so much for raising this issue!

Case sensitivity of identifiers such as module codes and lecture names was a notable design decision that we had to mull over. Initially, we believed cases insensitivity would be ideal for a smoother user experience. Ultimately, we chose to stick with case sensitivity as this would ensure our models are future-proof and flexible. We also referred to the conventions of Unix-based commands such as cd that are also case sensitive. Hence, case sensitivity would be ideal for ensuring that the product remains robust over time.

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 response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]