AY2223S2-CS2103T-W10-1 / tp

https://nus-cs2103-ay2223s2.github.io/tp/
Other
0 stars 4 forks source link

Update UML #156

Closed tingyic closed 1 year ago

tingyic commented 1 year ago

Update UML diagram to fit our app. Please help check if the diagrams are correct.

codecov[bot] commented 1 year ago

Codecov Report

Merging #156 (8ee5665) into master (88928c0) will increase coverage by 0.01%. The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #156      +/-   ##
============================================
+ Coverage     62.88%   62.89%   +0.01%     
+ Complexity      443      442       -1     
============================================
  Files            83       83              
  Lines          1611     1617       +6     
  Branches        174      175       +1     
============================================
+ Hits           1013     1017       +4     
- Misses          539      540       +1     
- Partials         59       60       +1     

see 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

tingyic commented 1 year ago

Can you perhaps show us some screenshots of how the DG UML diagrams look like now?

image

This is the updated ModelClassDiagram, for example.

tingyic commented 1 year ago

Forgot to export the edited PUML files into PNG, so they weren't displayed. My bad :(