Open nus-se-bot opened 2 years ago
Thank you for raising this issue.
We note that the domain name is treated as case-insensitive by browsers. However, browsers usually will convert the domain name to lowercase, and since the main use case of our app is to copy the full meeting link into the command box instead of typing it out manually, this is out of scope for our application at the current version. We may consider updating the regex to allow capital letters in the domain name in the future.
--
Bug: System does not accept uppercase letters in the URL's second-level domain
Description: The application does not allow me to enter uppercase letters in the second-level domains (i.e.
M1
inhttps://www.M1.com
). While it is true that it is always lowercase, there are occasions when users type in the URL themselves and they might uppercase it by habit (eg. typing www.Google.com). Browsers automatically make the letters lowercase so it will still be functional when using theopen
command.Steps to reproduce: Enter command
add n/M1 u/https://www.M1.com d/16-04-2022 1600 t/tag3 dur/2 m/1 r/Y t/tag1 t/tag2
Expected outcome: Either: (1) The system should accept it; or (2) The system should provide a descriptive message saying that the uppercase letter is not permitted
Actual outcome: The error
URLs should be a valid link, and it should not be blank
is given which does not guide users on how to fix it.[original: nus-cs2103-AY2122S2/pe-interim#1211] [original labels: severity.Low type.FunctionalityBug]