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

Update DG to include implementation details on Note Command #111

Merged
merged 8 commits into from
Oct 21, 2020

Conversation

wang-jun-hao
Copy link

No description provided.

* 'master' of https://github.com/AY2021S1-CS2103T-F13-3/tp:
  Remove wildcard import
  Add new line at EOF for MedicalNoteListTest
  change font size
  Remove debugging code from NoteCommand and Patient classes
  Handle exception for adding medical note with empty content in NoteCommandParser.java
  Update test in NoteCommandTest to use Model#resetAccessedPatient
  Add unit and integration tests for Date, NoteCommand, MedicalNoteList
  Update Patient equals method to include checking of medical notes store in patient
* 'master' of https://github.com/AY2021S1-CS2103T-F13-3/tp:
  Update DeveloperGuide.md
  Update diagrams to fit code conventions
  Update DG for optional fields and login functionality
* master:
  Update DeveloperGuide.md
  Update diagrams to fit code conventions
  Update DG for optional fields and login functionality
@wang-jun-hao wang-jun-hao added the type.Documentation Improvements or additions to documentation label Oct 21, 2020
@codecov-io
Copy link

codecov-io commented Oct 21, 2020

Codecov Report

Merging #111 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #111   +/-   ##
=========================================
  Coverage     68.07%   68.07%           
  Complexity      595      595           
=========================================
  Files            94       94           
  Lines          2055     2055           
  Branches        234      234           
=========================================
  Hits           1399     1399           
  Misses          579      579           
  Partials         77       77           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f78c2f...ad32208. Read the comment docs.

Copy link

@Wong-ZZ Wong-ZZ left a comment

Choose a reason for hiding this comment

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

lgtm

@Wong-ZZ Wong-ZZ merged commit ebc6b9a into AY2021S1-CS2103T-F13-3:master Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type.Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants