nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Small typos in sequence diagram and write-up for Group function #2328

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


In the second line, the execute("group c/Clan) is missing a " In the diagram, the parameter turns from "c/Clan" to "c/clan" when passed in the GroupCommandParser

Screenshot 2024-04-19 at 5.36.11 PM.png


[original: nus-cs2103-AY2324S2/pe-interim#1923] [original labels: type.DocumentationBug severity.VeryLow]

Cedricaca commented 4 months ago

Team's Response

execute has no "" in execute(m) as m refers to the m:Model, not a string.

parameter changes from c/Clan to c/clan as the parse turns it into lowercase as a feature due to categories being case insensitive.

Not a bug

Duplicate status (if any):

--