search

nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Program allows numbers to added in names. #3435

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

No details provided by bug reporter.

[original: nus-cs2103-AY2122S2/pe-interim#3506] [original labels: severity.Low type.FunctionalityBug]

bingsen0806 commented 2 years ago

Team's Response

The tester does not is not specific about whether he is referring to patron’s name or book names. Either way, it should be rejected.

For patron names, we don’t want to restrict names not having numbers in them as there might be cases where the patron’s name. The diagram below shows an example of number 16 being included in a name.

image.png

image source from newspaper

Furthermore, book names can contain numbers (For example, classics like November 9, 1984 by George Orwell contain numbers). Our users should have the flexibility in adding such patrons or books with numbers in name into our app.

Duplicate status (if any):

--