diff --git a/docs/DeveloperGuide.md b/docs/DeveloperGuide.md index 25054c5d0e1..414b8f4c506 100644 --- a/docs/DeveloperGuide.md +++ b/docs/DeveloperGuide.md @@ -188,8 +188,6 @@ the lifeline reaches the end of diagram. #### Design rationale {:.no_toc} -The `add-t` command was designed this way to ensure consistency with the previous `add` command in AB3. - **Aspect: Optional fields** * **Alternative 1:** Allow optional fields when adding tutor (e.g. user can omit phone number or email when adding a tutor). diff --git a/docs/diagrams/EditScheduleSequenceDiagram.puml b/docs/diagrams/EditScheduleSequenceDiagram.puml index db542c11595..20b4f13e912 100644 --- a/docs/diagrams/EditScheduleSequenceDiagram.puml +++ b/docs/diagrams/EditScheduleSequenceDiagram.puml @@ -6,10 +6,9 @@ box Logic LOGIC_COLOR_T1 participant ":LogicManager" as LogicManager LOGIC_COLOR participant ":AddressBookParser" as AddressBookParser LOGIC_COLOR participant ":EditSchedule\nCommandParser" as EditScheduleCommandParser LOGIC_COLOR -participant "e:EditSchedule\nDescriptor" as EditScheduleDescriptor LOGIC_COLOR participant "d:EditSchedule\nCommand" as EditScheduleCommand LOGIC_COLOR participant "r:CommandResult" as CommandResult LOGIC_COLOR -participant "s:Schedule" as EditedSchedule MODEL_COLOR +participant "e:Schedule" as EditedSchedule MODEL_COLOR end box box Model MODEL_COLOR_T1 @@ -32,13 +31,6 @@ deactivate EditScheduleCommandParser AddressBookParser -> EditScheduleCommandParser : parse("1 st/2023-09-15T09:00") activate EditScheduleCommandParser -create EditScheduleDescriptor -EditScheduleCommandParser -> EditScheduleDescriptor -activate EditScheduleDescriptor - -EditScheduleDescriptor --> EditScheduleCommandParser : e -deactivate EditScheduleDescriptor - create EditScheduleCommand EditScheduleCommandParser -> EditScheduleCommand activate EditScheduleCommand @@ -63,21 +55,21 @@ return create EditedSchedule EditScheduleCommand -> EditedSchedule ++ -return s -EditScheduleCommand -> Model ++ : hasSchedule(s) +return e +EditScheduleCommand -> Model ++ : hasSchedule(e) return EditScheduleCommand -> Model ++ : getAddressBook() return loop number of schedules in model - EditScheduleCommand -> EditedSchedule : isClashing(s) + EditScheduleCommand -> EditedSchedule : isClashing(e) activate EditedSchedule EditedSchedule --> EditScheduleCommand deactivate EditedSchedule end -EditScheduleCommand -> Model ++ : setSchedule(scheduleToEdit, s) +EditScheduleCommand -> Model ++ : setSchedule(scheduleToEdit, e) return EditScheduleCommand -> Model ++ : updateFilteredScheduleList() return diff --git a/docs/images/EditScheduleSequenceDiagram.png b/docs/images/EditScheduleSequenceDiagram.png index cd36fa1b3ef..7aa460e3359 100644 Binary files a/docs/images/EditScheduleSequenceDiagram.png and b/docs/images/EditScheduleSequenceDiagram.png differ