datn02 / pe

0 stars 0 forks source link

Quotation marks cannot be used in Notes #7

Open datn02 opened 2 years ago

datn02 commented 2 years ago

Command inputted: add "a" "SLAP stands for "Single level of Abstraction Principle""

When included quotation marks in notes content, it said that "missing arguments or invalid arguments", which is not true in this case.

There is no mention of anywhere users cannot include quotation marks in Notes content.

image.png

nus-se-script commented 2 years ago

Team's Response

Thank you for your report.

Due to how we use to keep track of our arguments in Terminus, double quotations is used as a delimiter to identify many arguments. Hence you can view the double quotes as a special syntax used in Terminus.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Quotes in question content

Unable to add quotes through cmd line but modifying main.json allows it to contain quotes.

(Cmd line)

image.png

(After modifying)

image.png

image.png


[original: nus-cs2113-AY2122S1/pe-interim#954] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for your report.

Due to how we use to keep track of our arguments in Terminus, double quotations is used as a delimiter to identify many arguments. Hence you can view the double quotes as a special syntax used in Terminus.

However, Terminus does not prevent you from loading double quotes and is intended by design.

We understand it could be confusing and the team recognize that this is a documentation bug, hence should be included in the documentation.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue type

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

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]