search

nus-cs2113-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Currency Abbreviation Accepts only Upper Case #3052

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

While I know this has been warned against doing in the UG and in the app itself, I do want to point out that the insisting the user types in uppercase could slightly hinder the user in using the application. Perhaps this needn't be case sensitive and the appropriate conversion be handled within the program?

image.png

[original: nus-cs2113-AY2021S1/pe-interim#3045]

tengkianen commented 3 years ago

Team's Response

The issue is not actually a bug, as we have specified the parameters constraints in the error message and even in the UG.

This 'issue' does not prove that the feature is less useful in any way.

Thank you for your feedback.

Duplicate status (if any):

--