search

zhengster / pe

0 stars 0 forks source link

Error should probably be thrown when the user doesn't space their commands correctly #3

Open zhengster opened 2 years ago

zhengster commented 2 years ago

Your UG specifies that the user input their "add doctor" command as follows: image.png

However, your implementation allows for spacing to occur. image.png

You could either fix this by specifying in the UG that lack of spaces are allowed, or you could fix the code itself to not accept improperly-spaced commands.

nus-se-script commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

I believe that small issues like inconsistent spacing is common for any typist. Our app is built to recognise these minor human errors and give a better user experience. The implementation was intended to accept inputs even if there were minor spacing issues. We did'nt find the need to include this in the UserGuide itself as its not really a feature that the user has to take note of, its just there to make the user experience better.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]