search

rageqqq / pe

0 stars 0 forks source link

Able to add invalid links to lesson info #4

Open rageqqq opened 3 years ago

rageqqq commented 3 years ago

image.png

I was able to add an invalid link to the lesson info. If the app is unable to open the link, it should not be able to store the link to prevent confusion to the user.

nus-pe-bot commented 3 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Links without http/ https should not be allowed to be stored

If there is a http/ https requirement, why allow links without http/ https to be stored?

image.png

Adding http:// allows the app to open the given website.

image.png

However, removing the http:// does not allow the app to open the website

[original: nus-cs2113-AY2021S2/pe-interim#950] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We agree that this is an issue. However, we believe that it causes only minor inconvenience to the user (since the only impact is not being able to open the link), hence we reduced the severity with low.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]

:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your reason]