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.

Sign up for GitHub

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

Jump to bottom

Add viewloan sequence diagram #69

Closed
narwhalsilent opened this issue Mar 22, 2024 · 0 comments · Fixed by #70
Closed

Add viewloan sequence diagram #69

narwhalsilent opened this issue Mar 22, 2024 · 0 comments · Fixed by #70
Assignees
@narwhalsilent
Labels
priority.High Must do type.Documentation All things related to documentation
Milestone
v1.2b

Comments

@narwhalsilent
Copy link

Add new sequence diagram to the new method viewloan.
@narwhalsilent narwhalsilent added priority.High Must do type.Documentation All things related to documentation labels Mar 22, 2024
@narwhalsilent narwhalsilent added this to the v1.2b milestone Mar 22, 2024
@narwhalsilent narwhalsilent self-assigned this Mar 22, 2024
narwhalsilent added a commit to narwhalsilent/tp that referenced this issue Mar 22, 2024
@narwhalsilent
Add viewloan sequence diagram
971e323 
Method currently violates Law of Demeter. Revision of the method may be necessary, in which case this diagram will need to be updated.

Fixes AY2324S2-CS2103T-W13-1#69.
@narwhalsilent narwhalsilent mentioned this issue Mar 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@narwhalsilent narwhalsilent

Labels
priority.High Must do type.Documentation All things related to documentation
Projects
None yet
Milestone
v1.2b
Development

Successfully merging a pull request may close this issue.

Add viewloan sequence diagram narwhalsilent/tp
1 participant
@narwhalsilent