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 Developer Guide with the latest documentation #58

Merged
merged 4 commits into from
Mar 29, 2024
Merged

Conversation

iamtr
Copy link

@iamtr iamtr commented Mar 29, 2024

This PR will update the Developer Guide, which consists of the latest documentation.

Copy link

codecov bot commented Mar 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.82%. Comparing base (7eb89ab) to head (f08a3e5).

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #58   +/-   ##
=========================================
  Coverage     74.82%   74.82%           
  Complexity      431      431           
=========================================
  Files            72       72           
  Lines          1394     1394           
  Branches        134      134           
=========================================
  Hits           1043     1043           
  Misses          314      314           
  Partials         37       37           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


Step 1. The user launches the application for the first time.

Step 2. The user executes an Add Command (e.g. 'add n\Alice ad\01/01/2022 dob\01/01/2002 ward\WA') to add a new patient to the address book.
Copy link

Choose a reason for hiding this comment

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

the add command here has some errors

  1. doesn't have the ic\ field
  2. ward\ should just be w\

n\: Indicates the name of the patient
ad\: Indicates the admission date of the patient
dob\: Indicates the date of birth of the patient
ward\: Indicates the ward of the patient is currently in
Copy link

Choose a reason for hiding this comment

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

Same here as above, ward\ should just be w\ and we should also include ic\

@iamtr iamtr merged commit c657a4b into master Mar 29, 2024
8 checks passed
Copy link

@ryanlimdx ryanlimdx left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@swtan346 swtan346 left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants