samuelim01 / pe

0 stars 0 forks source link

Add booking command rejects valid input #3

Open samuelim01 opened 10 months ago

samuelim01 commented 10 months ago

Description

According to the user guide,

The domain name must end with a domain label that is supported: gmail, yahoo, outlook, hotmail, icloud.

However, an email with the domain name yahoo.com.sg is not accepted by the application

Steps to reproduce

Expected outcome

Actual outcome

Platform

Screenshot(s)

image.png

nus-pe-bot commented 10 months ago

Team's Response

The issue is with the phrasing in the documentation as a valid domain considered by our team is yahoo.com and not yahoo.com.sg

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** Unfortunately, I disagree that this issue is `severity.VeryLow`. `severity.VeryLow` is for purely cosmetic issues that do not affect usage. In this case, it does affect the usage because what was specified to be supported (`yahoo` domains) was not supported in all cases. I believe this is not an issue of extreme user behavior, because many people do use `yahoo.com.sg`. Hence, I believe this should be of `severity.Low` instead.