nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Unable to see full phone number, when phone number too long. #3800

Open nus-se-script opened 2 months ago

nus-se-script commented 2 months ago

I got this error when i added a user with a very long phone number. image.png While it is unlikely to have a phone number that is that long, there should be a horizontal scrolling to see the full content of the person.


[original: nus-cs2103-AY2324S2/pe-interim#4799] [original labels: severity.Low type.FeatureFlaw]

jinhanfromNUS commented 2 months ago

Team's Response

Thank you pointing this out.

Our team has decided to put NotInScope as response.

We have also notified this issue under future enhancement:

image.png

Referencing the second point, we plan to adjust the range of acceptable length for phone numbers in the future enhancements.

Note: While response.NotInScope are not counted against the dev team, they can earn a small amount of consolation marks for the tester. The other three do not affect marks of either the dev team or the tester.

Duplicate status (if any):

--