search

LiuMC-SG / pe

0 stars 0 forks source link

Telehandle does not allow last underscore #17

Open LiuMC-SG opened 1 year ago

LiuMC-SG commented 1 year ago

image.png

Tele handle error

nus-pe-bot commented 1 year ago

Team's Response

Telegram handles cannot have a trailing underscore, as specified by our UG and Telegram's specifications.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The specific specification on telegram states this.

image.png

Although the actual telegram application does not allow for the last character to be an underscore, it should be specified directly within the UG since it is not explicitly stated within telegram specifications but is done on the backend side.

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** Based on their explanation, this should be a `type.DocumentationBug` instead since the validation checks work but is not shown in the UG specifically for last underline does not work as shown here. ![image.png](https://raw.githubusercontent.com/LiuMC-SG/pe/main/files/ab50bd81-441b-469a-9e12-8d581f1846c1.png)
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]