Merge pull request AY2425S1-CS2103-F09-2#248 from starbucksventi/upda…

…te-sequence-diagram Fix bug in Favourite Command sequence diagram
starbucksventi · Nov 12, 2024 · faa1cca · faa1cca
2 parents 27314d5 + 9e14a37
commit faa1cca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/docs/diagrams/FavouriteCommandSequenceDiagram.puml b/docs/diagrams/FavouriteCommandSequenceDiagram.puml
@@ -36,6 +36,9 @@ create FavouriteCommand
 FavouriteCommandParser -> FavouriteCommand
 activate FavouriteCommand
 
+FavouriteCommand --> FavouriteCommandParser
+deactivate FavouriteCommand
+
 FavouriteCommandParser --> AddressBookParser : f
 deactivate FavouriteCommandParser
 
@@ -46,6 +49,7 @@ AddressBookParser --> LogicManager
 deactivate AddressBookParser
 
 LogicManager -> FavouriteCommand : execute()
+activate FavouriteCommand
 
 create Person
 FavouriteCommand -> Person : new(other_fields_in_personToEdit, newTagSet_presence_of_FAVOURITE_TAG_toggled)

diff --git a/docs/images/FavouriteCommandSequenceDiagram.png b/docs/images/FavouriteCommandSequenceDiagram.png