ChrisHo1341 / pe

0 stars 0 forks source link

Tag command does not allow whitespaces #8

Open ChrisHo1341 opened 2 months ago

ChrisHo1341 commented 2 months ago

image.png

image.png

Tag does not allow white spaces under the t/ parameter, I think this should be specified in the UG since it is specified for KEYWORD

nus-pe-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Adding a tag does not allow white spaces

Problem: It was not mentioned in the UG that tag cannot allow white spaces

Screenshot 2024-04-19 164714.png


[original: nus-cs2103-AY2324S2/pe-interim#1848] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

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

image.png

We did mentioned that Tags should be alphanumeric, but we could've done better in being more specific.

image.png

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.Rejected`] - [x] I disagree **Reason for disagreement:** Even though small, the lack of specificity would result in a user not knowing why their input is being rejected and having to guess and figure out the proper input through their own manual testing, which does affect the user experience
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.DocumentationBug`] - [x] I disagree **Reason for disagreement:** This error could be prevented if it is just mentioned within the UG. It was mentioned for other commands, which makes it even harder for the user to figure out what is wrong with their input for the command