search

dayn-1 / pe

0 stars 0 forks source link

Add command error does not have "subjects" #10

Open dayn-1 opened 3 months ago

dayn-1 commented 3 months ago

image.png

As seen above, the command format makes no mention of "subject", though "subject" is mentioned in the UG.

soc-pe-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Inconsistent format in TutorRec and in UG

Upon executing the add command, the error message shows a format requires all fields p/PHONE, e/EMAIL, etc as compulsory, and only ap/ and t/ are optional. This is in contrast to the format put in the UG, which states that only name is compulsory and the rest are optional.

Application: image.png

UG: image.png

Steps to replicate:

  1. Try keying in the command: "add"

[original: nus-cs2103-AY2324S2/pe-interim#2805] [original labels: type.FunctionalityBug severity.Low]

Their Response to the 'Original' Bug

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

Agreed

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: The issues are not the same.

Issue #1 is about how the add command's format in the application does not have "subjects" as a parameter, though it is a parameter in implementation and in the UG. "Subjects" is excluded from the command format all together, not a discrepancy between compulsory or optional.

Issue #2 is about how the discrepancy in optional vs compulsory parameters between the application and the UG. The in-app add command format states that phone, email, etc are compulsory although it is actually optional in the UG and actual implementation.