johnrhimawan / pe

0 stars 0 forks source link

Unable to Add INVESTMENT_PLAN of Plan #8

Open johnrhimawan opened 2 years ago

johnrhimawan commented 2 years ago

image.png image.png

Reproduce:

add n/Susan Joe p/12345678 e/susan@gmail.com a/New York i/200000 m/10000 r/HIGH ip/ Plan c/CURRENT

I am not able to add an INVESTMENT_PLAN of Plan even though the user guide would suggest that this is possible since it ends with the word Plan.

nus-pe-bot commented 2 years ago

Team's Response

We would argue that this is more of a documentation bug, we implemented the feature with the assumption that ALL investments plan have a word in front of the word plan. Does not make sense to just input Plan.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: I believe that this is not a documentation bug. If we head over to the module website, we can see that a FunctionalityBug means that a functionality does not work as specified/expected. Since it was specified in the user guide that INVESTMENT_PLAN must end with the word Plan, then putting in the word Plan should be readily available for the application. Instead, it throws an error which might confuse the user since the user has followed the user guide accurately. Since the intended behavior is not fulfilled, it should be considered a FunctionalityBug. The severity should be Low since it hinders the user of the application even when following the user guide.