Kang-Quan / pe

0 stars 0 forks source link

Valid website unable to be added using edit command #4

Open Kang-Quan opened 1 year ago

Kang-Quan commented 1 year ago

tinyurl.com/catcher-web is a valid website but it is unable to be keyed in into the app.

Steps to replicate: 1. type edit 2 web/tinyurl.com/catcher-web

image.png

soc-pe-bot commented 1 year ago

Team's Response

Website must start with https or http as clearly stated in the User Guide and in the error message. From a user perspective, they are more likely to copy the website URL into the command box from the webpage as compared to typing the whole URL as some website URL can be quite long. Copying of the whole URL (normal usage) would include the https or http protocol.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: From your user guide, this application is made with CLI users in mind and specifically computer science students. Some students may be well aware of the company's career page and is able to accurately type it out without reference.

For convenience sake, modern browsers will not require you to include http and https as it will be auto included. Since it is CLI based and most students are using modern browsers, most of the users will intuitively exclude https/http protocol when typing. For example, will a user type https://facebook.com or just facebook.com. This might be a case of overzealous input validation. Since accepting links like facebook.com, careers.shopee.sg or github.com will not hinder operation of software. While rejecting them may annoy or slow down the speed of using the application.

image.png

It is a flaw that occasionally affects user which causes some inconvenience but they can continue to use the product hence given a medium severity tag.

image.png