Greg-Tan / pe

0 stars 0 forks source link

add command bank field does not allow for "-" characters which is possible in bank account numbers #2

Open Greg-Tan opened 2 months ago

Greg-Tan commented 2 months ago

bank account numbers could possibly contain "-" or other non alphanumeric characters sometimes which your app does not allow. the returned error message is also not very clear that only numbers are allowed

image.png

nus-se-script commented 2 months ago

Team's Response

Bank numbers are typically inputted without the "-".

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Response given is not true as banks like DBS and POSB has the format of 123-4-567890 for dbs and 123-45678-9 Additionally, the error message from the app does not mention what is wrong with the format that i inputted because technically 123-45678-9 does have a digit length of 9 and hence, making it hard for my to debug my input to know what I entered wrongly