search

aaronxujiachen / pe

0 stars 0 forks source link

ViewFinancialStatement feature should be made more secure #5

Open aaronxujiachen opened 10 months ago

aaronxujiachen commented 10 months ago

According to the UG, the correct format of viewing all financal statements should be "viewFinancialStatement".

Screenshot 2023-11-17 165848.png

However, it seems like the command does not consider the mixing of uppercase and lowercase.

Screenshot 2023-11-17 165823.png

nus-pe-bot commented 10 months ago

Team's Response

On the contrary, it was designed as such on purpose, so that casing was not a problem. In fact, this was explicitly mentioned on the CS2113 website.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]