Skip to content

Commit

Permalink
Merge pull request #259 from laney0808/fix-icon
Browse files Browse the repository at this point in the history
Change return value to true for hasPerson().
  • Loading branch information
laney0808 authored Apr 15, 2024
2 parents 51bd98d + b98c223 commit 6425be9
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 9 deletions.
6 changes: 0 additions & 6 deletions docs/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,6 @@ title: Developer Guide

--------------------------------------------------------------------------------------------------------------------

## **Acknowledgements**

* {list here sources of all reused/adapted ideas, code, documentation, and third-party libraries -- include links to the original source as well}

--------------------------------------------------------------------------------------------------------------------

## **Setting up, getting started**

Refer to the guide [_Setting up and getting started_](SettingUp.md).
Expand Down
2 changes: 1 addition & 1 deletion docs/diagrams/AddVisitCommandLogic.puml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ Visit --> AddVisitCommand : nric
deactivate Visit
AddVisitCommand -> Model : hasPerson(...)
activate Model
Model --> AddVisitCommand : false
Model --> AddVisitCommand : true
deactivate Model
AddVisitCommand -> Model : hasVisit(...)
activate Model
Expand Down
2 changes: 1 addition & 1 deletion docs/diagrams/DeleteSequenceDiagram.puml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ activate DeleteCommand

DeleteCommand -> Model : hasPerson(...)
activate Model
Model --> DeleteCommand : false
Model --> DeleteCommand : true
deactivate Model
DeleteCommand -> Model : deletePerson(...)
activate Model
Expand Down
2 changes: 1 addition & 1 deletion docs/diagrams/ReadCommandSequenceDiagram.puml
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ activate ReadCommand
ReadCommand -> Model : hasPerson(...)
activate Model

Model --> ReadCommand : false
Model --> ReadCommand : true
deactivate Model

ReadCommand -> Model : updateFilteredPersonList(...)
Expand Down
Binary file modified docs/images/DeleteInfoSequenceDiagram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/DeleteSequenceDiagram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified docs/images/ReadCommandSequenceDiagram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 6425be9

Please sign in to comment.