search

AngJunYang / pe

0 stars 0 forks source link

Multiple punctuations not allowed in addevent command #4

Open AngJunYang opened 1 year ago

AngJunYang commented 1 year ago

The app does not allow the event name D...laksa even though "D...laksa" contains only alphanumeric with punctuations.

image.png

nus-se-bot commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

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

Unable to enter punctuations in addevent command

addevent ev/D’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’’ from/01-05-2023 17:30 to/01-05-2023 19:30 addevent ev/D..…….……….……….……….………. from/01-05-2023 17:30 to/01-05-2023 19:30

The above test cases are not allowed in the app, even though they are alphanumeric characters with spaces.

image.png

image.png

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

Their Response to the 'Original' Bug

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

Thanks for providing us with your testing command for us to reproduce. We tried the symbols you used in the commands and realized that the symbols are not an ASCII character. In our developer guide, we mentioned that our application should and would minimally support ASCII characters.

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 reason]

## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]