nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

[DG] Special Characters Enhancement has too many overlaps, leading to lesser possible flagging of FFs #2122

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 months ago

image.png

While, its a valid enhancement, I do not believe this should be just one. It takes many code changes to several function to fix the many different fields that do not accept symbols.

For example, NAME should accept symbols is one enhancement on its own.

This prevents me from flagging out potential feature flaws for multiple fields.


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

simkianseng commented 5 months ago

Team's Response

Hi thanks for the catch. We believe that flagging these feature flaws will be of low severity, hence we have decided to classify this as low severity.

Duplicate status (if any):

--