nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Overzealous input validation for company field #2490

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

Description: The company input validation is overly strict and rejects inputs with symbols.

image.png

Company names potentially have symbols, and the application imposes unnecessary restrictions

Rationale: severity.Medium - Overly strict company validations prevent users from inputting accurate company names which might frustrate users.


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

theman-oj10 commented 6 months ago

Team's Response

We agree that this is a valid issue. However, company names with symbols are very rare, so we believe that our input validation is acceptable for the app to work 'reasonably well'. Hence, this has a lower priority than the work already done in v1.4, which is why we put it as NotInScope.

Duplicate status (if any):

--