zixin448 / pe

0 stars 0 forks source link

Unable to add professor to module after adding professor to contact book #10

Open zixin448 opened 2 years ago

zixin448 commented 2 years ago

Input

  1. add professor without m/ prefix (add o/professor n/prof p/99999999)
  2. in module view (particularly cs3223) -> add o/professor n/prof p/99999999 m/cs3223 Expected: add professor to module Actual: gives existing person error Issue: inconvenience to user to delete the contact and reenter

Screenshot 2022-04-16 at 3.03.55 PM.png

nus-se-bot commented 2 years ago

Team's Response

The add command is responsible for adding objects to the UniBook, not modification of existing objects. That would fall under edit command. Please refer to the User Guide for details on how to use the edit command.

Explanation: After adding a professor without the m/ prefix using add o/professor n/prof p/99999999, you have added this professor to the UniBook.

In order to assign him to a module (particularly cs3223), you would need to use the following edit command on the people view, edit [INDEX OF THE PROF] o/person nm/cs3223.

Not sure what severity to put actually since this isn't even a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]