search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incomplete Service Field Validation #3971

Open nus-se-script opened 15 hours ago

nus-se-script commented 15 hours ago

The User Guide states that the service field "Can contain any value except slash (/)", but this constraint is neither reflected in the Developer Guide's test cases nor mentioned in the system architecture diagrams. There appears to be no validation implementation for this documented constraint.

[original: nus-cs2103-AY2425S1/pe-interim#3924] [original labels: severity.Low type.DocumentationBug]

leslieyip02 commented 4 hours ago

Team's Response

It is true that this constraint is not reflected in our DG. We didn't feel the need to include these constraints within the current version of our DG because it is not meant to be an exhaustive description exactly how every piece of code in the app works. Instead, we believe the DG should reflect high-level design decisions. It serves as a starting point for developers to do their own testing and start developing.

Duplicate status (if any):

--