Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AB3: Tracing Code] Add more headers to section instructions #42

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
cb71de9
Edit LogicSequenceDiagram puml to show exact arguments
UdhayaShan1 Jul 7, 2024
6d49bce
Fix LogicSequenceDiagram by showing creation of parser
UdhayaShan1 Jul 7, 2024
16abf00
Add new edit sequence puml with MainWindow and model for future use
UdhayaShan1 Jul 7, 2024
ec55e13
Update tracingcode.md with h3 css style added
UdhayaShan1 Jul 7, 2024
005583c
Merge branch 'master' into tracing-code-add-headers
UdhayaShan1 Jul 7, 2024
464e91a
Edit arguments
UdhayaShan1 Jul 8, 2024
5ad51a5
Fix EditSequenceDiagramWithMainWindow png and remove html anchor
UdhayaShan1 Jul 13, 2024
9212c14
Edit LogicSequence puml to show execution and add annotated images
UdhayaShan1 Jul 13, 2024
a65f636
Add a logic sequence diagram only with mainwindow and logicmanager
UdhayaShan1 Jul 13, 2024
9f8f897
Add remaining headers and diagrams
UdhayaShan1 Jul 13, 2024
2990596
Update image
UdhayaShan1 Jul 13, 2024
4cfab7b
Merge branch 'master' into tracing-code-add-headers
UdhayaShan1 Jul 27, 2024
8fb8f61
Remove redundant lines and custom css
UdhayaShan1 Jul 27, 2024
5fc1be6
Updated tracing code high level sequence diagram description
UdhayaShan1 Jul 27, 2024
da6ae49
Removed MainWindow from puml to not overlap components
UdhayaShan1 Jul 27, 2024
3327fde
Misc change
UdhayaShan1 Jul 27, 2024
122ca7d
Add double quotes for puml method call arguments
UdhayaShan1 Jul 27, 2024
42c96b5
Add label content and change label names to "T<no>" in bold
UdhayaShan1 Jul 27, 2024
66cca5e
Making spacing consistent
UdhayaShan1 Jul 27, 2024
451b0fe
Update label content to be clearer
UdhayaShan1 Jul 27, 2024
e5e4331
Remove unnecessary spacing and fix typos
UdhayaShan1 Jul 27, 2024
e7db74d
Merge branch 'master' into tracing-code-add-headers
UdhayaShan1 Jul 27, 2024
ba92d2f
Added deletion to puml and misc change
UdhayaShan1 Jul 28, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix LogicSequenceDiagram by showing creation of parser
UdhayaShan1 committed Jul 7, 2024
commit 6d49bcee508ad5801bd9e744ecd505dd3658d683
Binary file added tutorials/images/tracing/LogicSequenceDiagram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
3 changes: 3 additions & 0 deletions tutorials/images/tracing/LogicSequenceDiagram.puml
Original file line number Diff line number Diff line change
@@ -12,6 +12,9 @@ activate logic
logic -> abp ++: parseCommand(edit 1 n/Alice Yeoh)
create ecp
abp -> ecp
activate ecp
ecp -> abp
deactivate ecp
abp -> ecp ++: parse(1 n/Alice Yeoh)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
abp -> ecp ++: parse(1 n/Alice Yeoh)
abp -> ecp ++: parse("1 n/Alice Yeoh")

Check other places as well.

create ec
ecp -> ec ++: 1, n/Alice Yeoh
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
27 changes: 27 additions & 0 deletions tutorials/images/tracing/LogicSequenceDiagramWithMainWindow.puml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
@startuml
!include ../style.puml
skinparam ArrowFontStyle plain

Participant ":MainWindow" as main LOGIC_COLOR
Participant ":LogicManager" as logic LOGIC_COLOR
Participant ":AddressBookParser" as abp LOGIC_COLOR
Participant ":EditCommandParser" as ecp LOGIC_COLOR
Participant "command:EditCommand" as ec LOGIC_COLOR

main -> logic : execute(edit 1 n/Alice Yeoh)
activate logic
logic -> abp ++: parseCommand(edit 1 n/Alice Yeoh)
create ecp
abp -> ecp
activate ecp
ecp -> abp
deactivate ecp
abp -> ecp ++: parse(1 n/Alice Yeoh)
create ec
ecp -> ec ++: 1, n/Alice Yeoh
ec --> ecp --
ecp --> abp --: command
abp --> logic --: command
logic --> main --: command

@enduml