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 documentation #198

Merged
merged 17 commits into from
Nov 9, 2020

Conversation

ZhangWanlin98
Copy link
Collaborator

  • Added Commands write up
  • Added Natural Date & Time section under command format
  • Updated FAQ with a link to Date & Time format
  • Added glossary
  • Updated command diagrams in user guide
  • Updated PPP

@ZhangWanlin98 ZhangWanlin98 self-assigned this Nov 8, 2020
@codecov-io
Copy link

codecov-io commented Nov 8, 2020

Codecov Report

Merging #198 (6a657c0) into master (6015043) will increase coverage by 6.19%.
The diff coverage is 36.36%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #198      +/-   ##
============================================
+ Coverage     74.78%   80.98%   +6.19%     
- Complexity      943     1037      +94     
============================================
  Files           125      125              
  Lines          2951     2982      +31     
  Branches        376      382       +6     
============================================
+ Hits           2207     2415     +208     
+ Misses          640      468     -172     
+ Partials        104       99       -5     
Impacted Files Coverage Δ Complexity Δ
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...a/seedu/address/logic/parser/ArgumentMultimap.java 100.00% <ø> (+14.28%) 9.00 <0.00> (+2.00)
...rc/main/java/seedu/address/model/ModelManager.java 98.27% <ø> (+9.48%) 57.00 <0.00> (+5.00)
src/main/java/seedu/address/model/UserPrefs.java 98.18% <ø> (+14.25%) 23.00 <0.00> (+5.00)
...ain/java/seedu/address/model/appointment/Time.java 95.23% <ø> (-0.42%) 14.00 <0.00> (-3.00)
...seedu/address/model/interval/TimeIntervalList.java 86.20% <0.00%> (-6.39%) 12.00 <0.00> (ø)
...rc/main/java/seedu/address/model/patient/Nric.java 66.66% <ø> (ø) 7.00 <0.00> (ø)
...address/storage/archive/CsvAppointmentArchive.java 92.85% <ø> (+8.92%) 18.00 <0.00> (+4.00)
...rc/main/java/seedu/address/ui/AppointmentCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...in/java/seedu/address/ui/AppointmentListPanel.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
... and 38 more

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 4c15b98...6a657c0. Read the comment docs.

Copy link
Collaborator

@YangYue128-helen YangYue128-helen left a comment

Choose a reason for hiding this comment

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

Other than a minor phrasing issue, LGTM!
We can see whether your images look ok after merging.

docs/UserGuide.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@JinHao-L JinHao-L left a comment

Choose a reason for hiding this comment

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

Other than some minor details, the rest LGTM 👍

docs/UserGuide.md Outdated Show resolved Hide resolved
docs/UserGuide.md Outdated Show resolved Hide resolved
docs/UserGuide.md Outdated Show resolved Hide resolved
@JinHao-L JinHao-L merged commit 846efaf into AY2021S1-CS2103T-T12-4:master Nov 9, 2020
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.

None yet

4 participants